Re: [PATCH] vhost-user-blk: Fix two resource leaks

2020-10-28 Thread AlexChen
On 2020/10/28 23:40, Raphael Norwitz wrote: > The change looks good but I'm not sure I'd call it resource leak in > either case since the failure case kills vhost-user-blk/scsi. In the > commit message maybe rather say "vhost-user-blk/scsi: fix broken error > handling for socket call"? > Thanks fo

Re: [PATCH] vhost-user-blk: Fix two resource leaks

2020-10-28 Thread Raphael Norwitz
The change looks good but I'm not sure I'd call it resource leak in either case since the failure case kills vhost-user-blk/scsi. In the commit message maybe rather say "vhost-user-blk/scsi: fix broken error handling for socket call"? On Wed, Oct 28, 2020 at 10:10 AM AlexChen wrote: > > When sock