Hi,
I'm sending a small patch, which should be fixing call of free() function upon 
invalid pointer.

Bugzilla record is here:
https://bugzilla.redhat.com/show_bug.cgi?id=498550

According to glibc example, original free() was placed correctly, but after 
quick peak to source code of glibc, I found out the new placement is actually 
correct.

Signed-off-by: Jan Zeleny <[email protected]>
---

--- fcoe-utils-1.0.7/fcoeadm.c  2009-03-27 21:40:22.000000000 +0100
+++ fcoe-utils-1.0.7/fcoeadm.c.free     2009-05-04 11:29:00.000000000 +0200
@@ -250,8 +250,8 @@ fcoeadm_find_fchost(char *ifname, char *
                        }
                        free(namelist[n]);
                }
+               free(namelist);
        }
-       free(namelist);
 
        return found;
 }
_______________________________________________
devel mailing list
[email protected]
http://www.open-fcoe.org/mailman/listinfo/devel

Reply via email to