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

Pino Toscano toscano.pino@tiscali.it
Fri Jul 20 16:55:00 GMT 2012


Hi,

Alle mercoledì 13 giugno 2012, Pino Toscano ha scritto:
> 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?)

Ping, about this and the other glibc patches I sent and that are 
awaiting review.

-- 
Pino Toscano
-------------- 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/20120720/83ec130e/attachment.sig>


More information about the Libc-alpha mailing list