Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <51254E0F.7010206@barfooze.de>
Date: Wed, 20 Feb 2013 23:28:31 +0100
From: John Spencer <maillist-musl@...fooze.de>
To: musl@...ts.openwall.com
Subject: Re: strcasestr.c

On 02/17/2013 08:04 PM, Rich Felker wrote:
> On Thu, Feb 14, 2013 at 10:23:49AM -0500, Rich Felker wrote:
>> On Thu, Feb 14, 2013 at 09:59:56AM -0500, Todd C. Miller wrote:
>>> When investigating using the musl strstr.c ofr OpenBSD I noticed
>>> that musl only has a stub for strcasestr() that calls strstr().

according to git log, this function dates back to the very first musl 
commit ever...

> Since strcasestr is nonstandard and not clearly specified,

it's so non-standard that even nobody uses it.
i looked up the usage of the function in codesearch.debian.net, and the 
only *user* (from all ~20K debian packages) of the function is gnu wget.
every other occurence of the function is from gnulib embedded into other 
packages.
so the current status is: gnulib includes the function, but nobody 
besides gnu wget uses it.

> Comments from anybody else?

given the above findings, we can just leave the function as-is (plus a 
comment that nobody uses it anyway) or remove it entirely.
in any case it doesnt make sense to put much work and especially much 
code into it.
if it's gonna be implemented "correctly" at all, it should be as slim as 
possible, in the order of 3-5 LOC.

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.