On 03/31/2015 10:52 PM, Alexey Kodanev wrote:
> Hi!
> On 03/31/2015 10:46 AM, Xing Gu wrote:
>> Signed-off-by: Xing Gu <gux.f...@cn.fujitsu.com>
>> ---
>> testcases/network/lib6/runcc.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/testcases/network/lib6/runcc.c b/testcases/network/lib6/runcc.c
>> index 91a0b16..3a89598 100644
>> --- a/testcases/network/lib6/runcc.c
>> +++ b/testcases/network/lib6/runcc.c
>> @@ -93,7 +93,7 @@ int runcc(char *tname, char *filename0, char *program)
>> cflags, filename0, filename);
>> es = system(cmd);
>> if (WEXITSTATUS(es) == 127) {
>> - tst_resm(TBROK, "can't run C compiler: \"%s\"", cmd);
>> + tst_resm(TCONF, "can't run C compiler: \"%s\"", cmd);
>> if (unlink(filename) < 0)
>> tst_resm(TWARN, "%s; unlink \"%s\" failed: %s", tname,
>> filename, strerror(errno));
>
> Why not 'tst_brkm(TCONF, ...)', do we need to proceed further?
>
The function "runcc" has created a file named "filename" before
executing "cmd" command, so I think we should delete this file
after tst_resm(TCONF,...).
Regards,
Xing Gu
> Thanks,
> Alexey
>
------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
