Re: [PATCH 01/41] usage.c: move format processing out of die_errno()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, May 02, 2016 at 11:26:02AM -0700, Junio C Hamano wrote:

> Caller supplied buffer would be the way to go when multiple threads
> could be showing errors and warnings, right?
> 
> It would not make too much of a difference for die(), though.

I think it can matter for die(). This is happening above the pluggable
die_routine() layer, so it's possible for an async task to call die()
which will end in pthread_exit(), while the main program is calling
die() to end in regular exit().

I imagine it's exceedingly unlikely, and of course both threads are
dying anyway, but it could result in some pretty weird memory errors.
It's probably worth taking precautions against, especially because it's
to easy to do so.

-Peff
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]