[libvirt] [PATCH] [libvirt-php] fix libvirt_stream_send/libvirt_stream_recv

Michal Privoznik mprivozn at redhat.com
Thu Mar 30 13:32:56 UTC 2017


On 03/30/2017 03:18 PM, Vasiliy Tolstov wrote:
> 2017-03-30 16:06 GMT+03:00 Michal Privoznik <mprivozn at redhat.com>:
>> Frankly, I've no idea whether this should be here. I feel like no because
>> VIRT_ZVAL_STRINGL should not look into the recv_buf at any position greater
>> than retval. That is why we give it the length of the string after all. Can
>> you please check whether your case still works after this line is removed?
>
>
> I'm try raw images and  recv_buf[retval] = '\0'; does not bring any
> problems (md5sum on source and dest equal with and without this).
> But can you look into send function in this function we convert zval
> to C string. As i understand leading zero dropped in this case ?
>

I guess so. So if you want to drop the line feel free to propose the 
patch. If you don't feel like touching the code - that is okay too.

Michal




More information about the libvir-list mailing list