[CRIU] [PATCH 0/6] Fixes and tests for dumpable flag (in particular handling fs.suid_dumpable=2)

Pavel Emelyanov xemul at parallels.com
Tue Jun 24 11:45:40 PDT 2014


On 06/18/2014 07:33 AM, Filipe Brandenburger wrote:
> Hi Pavel,
> 
> I hope these are still in time for the 1.3.0-rc2 release tomorrow.
> 
> I created two test cases, one basic test to confirm that restoring the dumpable
> flag works in the simple case and another complex one that does some gymnastics
> to get a process with the dumpable flag disabled (set to either 0 or 2
> depending on fs.suid_dumpable) dumped and restored by criu.
> 
> I'm also sending the revert for Saied's temporary workaround and two minor
> cleanups in .gitignore files.
> 
> Cheers!
> Filipe
> 
> 
> Filipe Brandenburger (6):
>   zdtm: add new dumpable01 test to check that dumpable flag is preserved
>   Revert "pie: A quick workaround for PR_SET_DUMPABLE == 2 restore
>     error."
>   restore: preserve dumpable flag when it is set to 2
>   zdtm: add new dumpable02 test to check that dumpable flag set to 0 or
>     2 works
>   zdtm: add missing entries to test/zdtm/.gitignore
>   git: add /dev to test/.gitignore
> 
>  pie/restorer.c                     |  33 ++++--
>  test/.gitignore                    |   1 +
>  test/zdtm.sh                       |   2 +
>  test/zdtm/.gitignore               |   4 +
>  test/zdtm/live/static/Makefile     |   2 +
>  test/zdtm/live/static/dumpable01.c |  48 +++++++++
>  test/zdtm/live/static/dumpable02.c | 207 +++++++++++++++++++++++++++++++++++++
>  7 files changed, 291 insertions(+), 6 deletions(-)
>  create mode 100644 test/zdtm/live/static/dumpable01.c
>  create mode 100644 test/zdtm/live/static/dumpable02.c
> 

Set applied, thanks everyone!


More information about the CRIU mailing list