[flashrom] [PATCH] Modify strnlen() check for old OSes to check its presence directly

Stefan Tauner stefan.tauner at alumni.tuwien.ac.at
Sat Jan 23 18:12:08 CET 2016


On Mon, 4 Jan 2016 13:14:03 +0300
Andrey Korolyov <andrey at xdel.ru> wrote:

> >
> > thanks for the patch. Did you notice any breakage in the wild? I
> > hesitate to include your change because that could actually break some
> > configurations: in case HAVE_STRNLEN is not defined but strnlen is
> > provided nonetheless linking would fail. Actually I'd be surprised if it
> > would not make problems with other libc implementations.
>
> NetBSD before 6.0 is definitely the one which breaks here, as well as
> other late adopters of srtnlen(), like FreeBSD before 8.0. Please feel
> free to NAK the patch if you think that the support of ancient libcs
> is not something to worry about (and old BSDs are definitely not LTS
> distros with ten-year support span).

Thanks for the explanation. I have merged the respective change now in
r1917 because I could not find any problems with it on any of our
buildbot tests. Hopefully no one else will either :)

-- 
Kind regards/Mit freundlichen Grüßen, Stefan Tauner




More information about the flashrom mailing list