[PATCH 3/4] torture: add FSRVP message sequence timeout test

Michael Adam obnox at samba.org
Wed Jun 18 06:07:51 MDT 2014


On 2014-06-18 at 14:14 +0200, David Disseldorp wrote:
> On Wed, 18 Jun 2014 14:00:35 +0200, Michael Adam wrote:
> 
> > > +		/* only need to delete if create process didn't timeout */
> > > +		if (TEST_FSRVP_TOUT_NONE) {  
> > 
> > This looks strange.
> > Shouldn't it read "if (i == TEST_FSRVP_TOUT_NONE)" or similar?
> 
> Yes, you're right. Thanks for the feedback!

Sure. Looking at the other patches as well.
That was as far as I got before spotting s/th.
More later.

Cheers - Michael

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20140618/d25961be/attachment.pgp>


More information about the samba-technical mailing list