In confstr01.c,  it's wrong that it reset the buffer contents to NULL
before freeing the memory allocated.


Signed-off-by: Jin Bing Guo <[EMAIL PROTECTED]>
-----------
--- cvs_ltp.orig/testcases/kernel/syscalls/confstr/confstr01.c  2003-03-05
02:34:03.000000000 +0800
+++ cvs_ltp/testcases/kernel/syscalls/confstr/confstr01.c       2008-09-17
15:46:22.000000000 +0800
@@ -127,10 +127,10 @@ int main()
                                break;
                        }

-                       /* Reset the buffer contents to NULL */
-                       buf = '\0';
                        /* Free the memory allocated for config. name */
                        free(buf);
+                       /* Reset the buffer contents to NULL */
+                       buf = '\0';
                } else {
                        if (!buf) {
                                if ((!strcmp("XXX5_MYBIG_VERBIG_MYFLAGS",


(See attached file: Fix_free_error_confstr01.patch)



Best regards!

Jin Bing Guo  郭晋兵

Linux for System p Test
IBM China Systems & Technology Laboratory in Beijing
Tel:   +86-10-82454439
Email: [EMAIL PROTECTED]
-------------------------------------
"Do today what others won't so tomorrow you do what other's can't"

Attachment: Fix_free_error_confstr01.patch
Description: Binary data

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to