Re: [SCM] Samba Shared Repository - branch master updated -release-4-0-0alpha7-1431-g1b2c65f

[Tim Prouty]

On May 4, 2009, at 12:07 PM, Volker Lendecke wrote:

> On Mon, May 04, 2009 at 11:58:19AM -0500, Tim Prouty wrote:
> > -               if (fnum1 != -1) cli_close(cli1, fnum1);
> > -               if (fnum2 != -1) cli_close(cli2, fnum2);
> > +               cli_close(cli1, fnum1);
> > +               cli_close(cli2, fnum2);
>
> I think that's wrong. We need a (fnumx 1= (uint16_t)-1).

I was just mirroring the other usage in the file as the quick warning  
fix.  It seems a little bit strange to me to make an fd uint, when the  
standard api uses -1 to indicate error.

-Tim