Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 18 Mar 2012 22:17:24 -0400
From:      Jason Hellenthal <jhellenthal@dataix.net>
To:        Eitan Adler <eadler@freebsd.org>
Cc:        svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, svn-src-stable-8@freebsd.org
Subject:   Re: svn commit: r233147 - stable/8/lib/libc/string
Message-ID:  <20120319021724.GC13456@DataIX.net>
In-Reply-To: <CAF6rxgkhiznBRU04%2Bc%2BDPdBj9Omg03SqeSd9-0Ct-FfxAVm95g@mail.gmail.com>
References:  <201203190127.q2J1RtgB044557@svn.freebsd.org> <20120319015203.GB13456@DataIX.net> <CAF6rxgkhiznBRU04%2Bc%2BDPdBj9Omg03SqeSd9-0Ct-FfxAVm95g@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help


On Sun, Mar 18, 2012 at 09:58:12PM -0400, Eitan Adler wrote:
> On Sun, Mar 18, 2012 at 9:52 PM, Jason Hellenthal
> <jhellenthal@dataix.net> wrote:
> >
> >
> > On Mon, Mar 19, 2012 at 01:27:55AM +0000, Eitan Adler wrote:
> >> Author: eadler
> >> Date: Mon Mar 19 01:27:55 2012
> >> New Revision: 233147
> >> URL: http://svn.freebsd.org/changeset/base/233147
> >>
> >> Log:
> >>   MFC r232503:
> >>       POSIX mandates that swab do nothing when len < 0
> >>
> >>   PR:         140690
> >>   Approved by:        cperciva
> >>
> >> Modified: stable/8/lib/libc/string/swab.c
> >> ==============================================================================
> >> --- stable/8/lib/libc/string/swab.c   Mon Mar 19 01:27:30 2012        (r233146)
> >> +++ stable/8/lib/libc/string/swab.c   Mon Mar 19 01:27:55 2012        (r233147)
> >> @@ -45,6 +45,8 @@ swab(const void * __restrict from, void
> >>       int n;
> >>       char *fp, *tp;
> >>
> >> +     if (len <= 0)
> >> +             return;
> >
> > Does this not test to see if it is also equal to 0(zero) ?
> >
> > If I understand the above statement "POSIX mandates that swab do nothing
> > when len < 0" then the above code should be exactly that ... and not
> > testing whether it is equal to zero...
> 
> If the code doesn't check for len <= 0 then it will do something.
> 

To my understanding of the specification it should only return if len is
negative... 0 is not a negative number.

-- 
;s =;



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20120319021724.GC13456>