[CRIU] test/zdtm/cwd00: add error checking to cleanup code

Pavel Emelyanov xemul at parallels.com
Wed Aug 7 04:45:27 EDT 2013


On 08/07/2013 12:08 PM, Andrew Vagin wrote:
> Ack for the last five patches. Thanks.
> 
> Acked-by: Andrew Vagin <avagin at parallels.com>
> 
> On Tue, Aug 06, 2013 at 04:48:23PM -0700, Yicheng Qin wrote:
>> From: Yicheng Qin <yichengq at google.com>
>> Subject: [PATCH] test/zdtm/cwd00: add error checking to cleanup code
>>
>> The checking helps debug unexpected failures.
>>
>> The patch is useful to debug test failure. Chdir() may meet failure
>> if the process does not have enough permissioin. Then, it cannot
>> rename in-progress output file to output file. Finally, test script
>> cannot catch 'PASS' flag from the output file and record it as
>> failure. As the result, the test fails without any error message.
>>
>> Signed-off-by: Yicheng Qin <yichengq at google.com>

Applied


More information about the CRIU mailing list