[libvirt] [PATCH] virFDStreamThread: Make sure we won't exceed @length

[Jim Fehlig]

On 06/05/2017 06:53 AM, Michal Privoznik wrote:
> There's a problem with current streams after I switched them from
> iohelper to thread implementation. Previously, iohelper made sure
> not to exceed specified @length resulting in the pipe EOF
> appearing at the exact right moment (the pipe was used to tunnel
> the data from the iohelper to the daemon). Anyway, when switching
> to thread I had to write the I/O code from scratch. Whilst doing
> that I took an inspiration from the iohelper code, but since the
> usage of pipe switched to slightly different meaning, there was
> no 1:1 relationship between the codes.
> 
> Moreover, after introducing VIR_FDSTREAM_MSG_TYPE_HOLE, the
> condition that should made sure we won't exceed @length was
> completely wrong.
> 
> The fix is to:
> 
> a) account for holes for @length
> b) cap not just data sections but holes too (if @length would be
> exceeded)
> 
> For this purpose, the condition needs to be brought closer to the
> code that handles holes and data sections.

For the record, this fixes the libvirt-tck failures I mentioned. Thanks!

Regards,
Jim