Rishi, Would you like to comment more on this Patch ??
--Subrata On Mon, 2008-01-21 at 16:33 +0100, Patrick Kirsch wrote: > Hi, > I'm sure that is a beauty fix, decide if you want to circumvent the > warning. > Affected are the files pidns01.c, pidns03.c, shmnstest.c, the patch sets > only the return statement. > > Regards, > plain text document attachment > (fix_control_reaches_end_of_non-void_function) > diff -pur ltp-full-20071231.orig/testcases/kernel/containers/pidns/pidns01.c > ltp-full-20071231/testcases/kernel/containers/pidns/pidns01.c > --- ltp-full-20071231.orig/testcases/kernel/containers/pidns/pidns01.c > 2008-01-01 12:44:42.000000000 +0100 > +++ ltp-full-20071231/testcases/kernel/containers/pidns/pidns01.c > 2008-01-21 16:18:37.783853000 +0100 > @@ -101,7 +101,7 @@ int child_fn1(void *ttype) > tst_resm(TFAIL, "FAIL: Got unexpected result of" > " cpid=%d ppid=%d\n", cpid, ppid); > } > - tst_exit(); > + return 1; > } > > /*********************************************************************** > Nur in ltp-full-20071231/testcases/kernel/containers/pidns: pidns01.c.orig. > diff -pur ltp-full-20071231.orig/testcases/kernel/containers/pidns/pidns03.c > ltp-full-20071231/testcases/kernel/containers/pidns/pidns03.c > --- ltp-full-20071231.orig/testcases/kernel/containers/pidns/pidns03.c > 2008-01-01 12:44:42.000000000 +0100 > +++ ltp-full-20071231/testcases/kernel/containers/pidns/pidns03.c > 2008-01-21 16:19:06.372251000 +0100 > @@ -164,7 +164,7 @@ child_fn(pid_t Ppid) > closedir(d); > } > > - cleanup(); > + return 1; > } > > /* > Nur in ltp-full-20071231/testcases/kernel/containers/pidns: pidns03.c.orig. > diff -pur > ltp-full-20071231.orig/testcases/kernel/containers/sysvipc/shmnstest.c > ltp-full-20071231/testcases/kernel/containers/sysvipc/shmnstest.c > --- ltp-full-20071231.orig/testcases/kernel/containers/sysvipc/shmnstest.c > 2008-01-01 12:44:42.000000000 +0100 > +++ ltp-full-20071231/testcases/kernel/containers/sysvipc/shmnstest.c > 2008-01-21 16:19:50.000574000 +0100 > @@ -69,7 +69,7 @@ int check_shmid(void *vtest) > write(p2[1], "exists", 7); > shmctl(id, IPC_RMID, NULL); > } > - tst_exit(0); > + return 0; > } > > #define UNSHARESTR "unshare" > @@ -134,5 +134,5 @@ int main(int argc, char *argv[]) > > /* destroy the key */ > shmctl(id, IPC_RMID, NULL); > - tst_exit(0); > + return 0; > } > ------------------------------------------------------------------------- > This SF.net email is sponsored by: Microsoft > Defy all challenges. Microsoft(R) Visual Studio 2008. > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ > _______________________________________________ Ltp-list mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/ltp-list ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
