tridge@samba.org schrieb:
> 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


yep, sorry I thought data_blob has void *, I'll review my last commits and change the casts to uint8_t *



--
metze

Stefan Metzmacher www.samba.org