On Tue, Jun 19, 2018 at 11:01:00AM +0200, Marc-André Lureau wrote: > Hi > > On Tue, Jun 19, 2018 at 8:23 AM, Gerd Hoffmann <kra...@redhat.com> wrote: > >> @@ -237,7 +251,7 @@ static int vhost_user_read(struct vhost_dev *dev, > >> VhostUserMsg *msg) > >> error_report("Failed to read msg header." > >> " Size %d exceeds the maximum %zu.", msg->hdr.size, > >> VHOST_USER_PAYLOAD_SIZE); > >> - goto fail; > >> + return -1; > > > > Hmm? Looks like a pointless indirection ... > > Isn't it removed by the patch? :)
Ah, right. Sorry about the noise. cheers, Gerd