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

[Pino Toscano]

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

hurd_sendto.diff
Description: Text Data

signature.asc
Description: This is a digitally signed message part.