Hi,

I'm guessing it makes more sense to exit, just like it does if there's not
a 'nobody' password entry. In either case, it doesn't make sense to print a
warning and then immediately dereference a NULL pointer.


--- testcases/kernel/syscalls/creat/creat08.c.orig      2007-10-18 
17:33:49.000000000 -0500
+++ testcases/kernel/syscalls/creat/creat08.c   2007-10-18 17:34:28.000000000 
-0500
@@ -131,11 +131,13 @@ int main (int ac, char **av)
                 * Get the group IDs of group1 and group2. 
                 */
                if ((group = getgrnam("nobody")) == NULL) {
-                       tst_resm(TINFO, "nobody not in /etc/group");
+                       tst_resm(TBROK, "nobody not in /etc/group");
+                       tst_exit();
                }
                group1_gid = group->gr_gid;
                if ((group = getgrnam("bin")) == NULL) {
-               tst_resm(TINFO, "bin not in /etc/group");
+                       tst_resm(TBROK, "bin not in /etc/group");
+                       tst_exit();
                }
                group2_gid = group->gr_gid;
 

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to