[illumos-Advocates] RTI: 1120 Use real file descriptors for smbfs named pipes.
Garrett D'Amore
garrett at damore.org
Thu Jun 30 12:44:39 PDT 2011
You need to list your reviewers. (It was reviewed, wasn't it? I sort
of recall looking at this, but I'm not sure if I blessed it. I do see
some potential issues, like snprintf()'s return code not being checked
or cast to void, which should throw lint warnings...)
Otherwise I think its ok.
- Garrett
On Thu, 2011-06-30 at 15:36 -0400, Gordon Ross wrote:
> Final webrev:
> http://www.yalms.org/cr/illumos-1120/
>
> outgoing, pbchk, mail_msg (attached)
> The mail_msg has some noise due to:
> https://www.illumos.org/issues/1078
> but I don't see any "real" problems there.
>
> This is most effectively tested by the (coming) changes for 1121,
> which we have in the field. That's going fine.
>
> Gordon
> _______________________________________________
> Advocates mailing list
> Advocates at lists.illumos.org
> http://lists.illumos.org/m/listinfo/advocates
More information about the Advocates
mailing list