Marc-André Lureau <marcandre.lur...@gmail.com> writes: > Hi > > On Mon, Feb 29, 2016 at 7:40 PM, Markus Armbruster <arm...@redhat.com> wrote: >> Yes, the chardev is commonly useless after we read a bad version from >> it, but destroying it is inappropriate anyway: the user created it, so >> the user should be able to hold on to it as long as he likes. We >> don't destroy it on other errors. Screwed up in commit 5105b1d. >> >> Stop reading instead. >> >> Also note QEMU's behavior in ivshmem-spec.txt. >> >> Signed-off-by: Markus Armbruster <arm...@redhat.com> > > Interestingly enough, it seems both smartcard "passthru" and usb > redirect do chr_delete(). It would a nice follow up to standardize > this there too.
Wasn't aware of them. I'll see what I can do. > Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com> Thanks!