Date: Mon, 17 Jul 2000 12:51:42 -0700 (PDT) From: John Baldwin <jhb@FreeBSD.org> To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sbin/i386/fdisk fdisk.c Message-ID: <200007171951.MAA48590@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
jhb 2000/07/17 12:51:42 PDT
Modified files:
sbin/i386/fdisk fdisk.c
Log:
- Don't try to free mboot.bootinst before it has been allocated. If, for
some reason, mboot.bootinst is not initialized to NULL at the beginning
of the program, then the last commit to this would try to free whatever
bogus address is in it.
- Restore the behavior of free()'ing the mboot.bootinst buffer after we
abuse it to determine the sector size of the disk (as clearly noted in
the comments). Properly fix the double free() bug by setting the pointer
to NULL after we free it.
Revision Changes Path
1.43 +3 -3 src/sbin/i386/fdisk/fdisk.c
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200007171951.MAA48590>