[PATCH,HURD] sendto: do not crash when addr is NULL

Pino Toscano toscano.pino@tiscali.it
Wed Jun 13 21:15:00 GMT 2012


Hi,

Alle martedì 12 giugno 2012, Roland McGrath ha scritto:
> That's almost right, but you should avoid calling
> __mach_port_deallocate if APORT is MACH_PORT_NULL.  Also, avoid
> shadowing the local variable ERR with one of the same name in the
> subfunction.

Hm ok, I saw other cases of shadowing by internal subroutines, and 
assumed it was okay (since inside them you want to refer to the local 
"err" instead of the outermost one.

Thanks for the further review, patch attached.
(I would use the non-ChangeLog lines of the patch header as commit log, 
would that be okay?)

-- 
Pino Toscano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: hurd_sendto.diff
Type: text/x-patch
Size: 3137 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20120613/cf1dca0e/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part.
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20120613/cf1dca0e/attachment.sig>


More information about the Libc-alpha mailing list