Hi!
> Following patch adds return 0; at the end of main() in the pidns17.c (and also
> removes some spurious code comments). This patch fixes compiler warning which
> some builds systems treats as error and thuss breaks package building.

Uff, there is one more missing return in child_fn(), please apply this patch.

Signed-off-by: Cyril Hrubis [email protected]

-- 
Cyril Hrubis
[email protected]
Index: ltp-full-20100228/testcases/kernel/containers/pidns/pidns17.c
===================================================================
--- ltp-full-20100228.orig/testcases/kernel/containers/pidns/pidns17.c
+++ ltp-full-20100228/testcases/kernel/containers/pidns/pidns17.c
@@ -111,6 +111,8 @@ int child_fn(void *arg)
 
 	/* cleanup and exit */
 	CLEANUP();
+
+	return 0;
 }
 
 /***********************************************************************
@@ -141,7 +143,9 @@ int main(int argc, char *argv[])
 
 	/* cleanup and exit */
 	CLEANUP();
-}	/* End main */
+
+	return 0;
+}
 
 /*
  * cleanup() - performs all ONE TIME cleanup for this test at
@@ -149,8 +153,6 @@ int main(int argc, char *argv[])
  */
 void cleanup()
 {
-	/* Clean the test testcase as LTP wants*/
 	TEST_CLEANUP;
 	tst_exit();
-	/* NOTREACHED */
 }
------------------------------------------------------------------------------
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to