Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20120815143640.GM27715@brightrain.aerifal.cx>
Date: Wed, 15 Aug 2012 10:36:40 -0400
From: Rich Felker <dalias@...ifal.cx>
To: musl@...ts.openwall.com
Subject: Re: Todo for release?

On Wed, Aug 15, 2012 at 03:32:57PM +0200, Szabolcs Nagy wrote:
> * Szabolcs Nagy <nsz@...t70.net> [2012-08-15 12:20:29 +0200]:
> > the main justification i see is that
> > we already support bsd err and warn
> > apis which are required to print
> > the __progname as well
> > (currently they don't and actually
> > a simple warn("hi"); segfaults here
> > with musl but i havent investigated
> > it)
> 
> it seems warn(0) and err(1,0) segfault
> (they should handle fmt==0 before passing
> it to vfprintf)
> and they do not print the ': ' nor the
> __progname

Thanks for the report. This should be easy to fix. By the way, is it
worth making these functions take a lock on the file for the whole
operation (to make it atomic)? I'm leaning towards no, since they seem
to only be used in legacy junk that's all single-threaded anyway.

Rich

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.