Date: Wed, 18 Aug 2010 19:06:26 +0200 (CEST) From: Oliver Fromme <olli@lurza.secnetix.de> To: freebsd-geom@FreeBSD.ORG, bug-followup@FreeBSD.ORG, walter@pelissero.de Subject: Re: kern/149762: volume labels with rogue characters Message-ID: <201008181706.o7IH6Q3q042332@lurza.secnetix.de> In-Reply-To: <201008181650.o7IGo3vu096176@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Oliver Fromme wrote:
> I have reviewed the patch (the second one from the followup),
> and it looks good to me, except for a few minor style(9)
> issues, and one type problem: By default, char is unsigned,
^^^^^^^^^^^^^^^^^^
> so the comparisons p <= ' ' and p > '~' probably don't do
> what you expect.
I'm sorry, I meant signed, of course.
You need to specify "unsigned char" if you want it to be unsigned.
Best regards
Oliver
--
Oliver Fromme, secnetix GmbH & Co. KG, Marktplatz 29, 85567 Grafing b. M.
Handelsregister: Registergericht Muenchen, HRA 74606, Geschäftsfuehrung:
secnetix Verwaltungsgesellsch. mbH, Handelsregister: Registergericht Mün-
chen, HRB 125758, Geschäftsführer: Maik Bachmann, Olaf Erb, Ralf Gebhart
FreeBSD-Dienstleistungen, -Produkte und mehr: http://www.secnetix.de/bsd
"C is quirky, flawed, and an enormous success."
-- Dennis M. Ritchie.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201008181706.o7IH6Q3q042332>