svn commit: samba r3971 - in branches/SAMBA_4_0/source: client libcli/auth libcli/raw ntvfs/print passdb smb_server torture/basic

tridge at samba.org tridge at samba.org
Thu Nov 25 21:57:39 GMT 2004


Andrew,

 > >  			s = va_arg(ap, char *);
 > > -			pointers[i].data = s;
 > > +			pointers[i].data = (void *)s;
 > >  			pointers[i].length = strlen(s)+1;
 > >  			head_size += pointers[i].length;
 > >  			break;
 > 
 > a cast to void doesn't quite seem right here.  DATA_BLOB.data is
 > uint8_t, so shouldn't this be a uint8_t cast?  

you're right about this one

 > > -	pstring param, data;
 > > +	uint8_t param[1024], data[1024];
 > >  	NTSTATUS status;
 > >  
 > >  	memset(data, 0, sizeof(data));
 > 
 > I'm worried these changes are just going to make it harder to find and
 > eliminate pstrings from the source. 

In this case the use of pstring was wrong in the first place, as these
are binary buffers that are never used as strings. So I think this
change is reasonable.

Cheers, Tridge


More information about the samba-technical mailing list