On Mon, 26 Dec 2005 17:19:20 +0100
Håkon Løvdal <Håkon Løvdal <[EMAIL PROTECTED]>> wrote:

 | There is no point in the last "if (pth)" test; if pth is NULL the
 | program will crash
 | at the pth->Reserved2 reference. On the question of whether passing NULL
 | to a function like this should be valid or not I tend to think it
 | should not, and therefore
 | suggest adding PED_ASSERT(pth != NULL, return) at the beginning.
That thing slipped my attention.
Thank you, Håkon.

~ Leslie


-- 
PGP-KID: 0x52D70289

Attachment: pgpCkUIKbqoui.pgp
Description: PGP signature

_______________________________________________
Bug-parted mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/bug-parted

Reply via email to