On Tue, Oct 04, 2011 at 06:41:52PM +0900, Kyungmin Park wrote: > On Tue, Oct 4, 2011 at 6:08 PM, larsi <la...@atlantis.wh2.tu-dresden.de> > wrote: > > - free(ubifs_sb->s_fs_info); > > - free(ubifs_sb); > > + if (ubifs_sb != null) {
Should be NULL. > > + free(ubifs_sb->s_fs_info); > > + free(ubifs_sb); > > + } > Which statement is problem? Basically free() check the null address. > so If ubifs_sb->s_fs_info doesn't have value its skipped. and ubifs_sb > is similar. The problem is the first call to free() above. If ubifs_sb is null, then free(ubis_sb->...) will be a null pointer dereference. Bye for now, -- Laurence Withers, <lwith...@guralp.com> http://www.guralp.com/ Direct tel:+447753988197 or tel:+443333408643 Software Engineer General support queries: <supp...@guralp.com> CMG-DCM CMG-EAM CMG-NAM _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot