Re: [PATCH 0/8] Coverity fixes for vchan-socket-proxy

2020-06-14 Thread Jason Andryuk
On Sat, Jun 13, 2020 at 8:40 AM Marek Marczykowski-Górecki wrote: > > On Wed, May 27, 2020 at 10:59:55PM -0400, Jason Andryuk wrote: > > On Mon, May 25, 2020 at 6:36 PM Jason Andryuk wrote: > > > > > > On Sun, May 24, 2020 at 10:50 PM Jason Andryuk wrote: > > > > > > > > This series addresses so

Re: [PATCH 0/8] Coverity fixes for vchan-socket-proxy

2020-06-13 Thread Marek Marczykowski-Górecki
On Wed, May 27, 2020 at 10:59:55PM -0400, Jason Andryuk wrote: > On Mon, May 25, 2020 at 6:36 PM Jason Andryuk wrote: > > > > On Sun, May 24, 2020 at 10:50 PM Jason Andryuk wrote: > > > > > > This series addresses some Coverity reports. To handle closing FDs, a > > > state struct is introduced t

Re: [PATCH 0/8] Coverity fixes for vchan-socket-proxy

2020-05-27 Thread Jason Andryuk
On Mon, May 25, 2020 at 6:36 PM Jason Andryuk wrote: > > On Sun, May 24, 2020 at 10:50 PM Jason Andryuk wrote: > > > > This series addresses some Coverity reports. To handle closing FDs, a > > state struct is introduced to track FDs closed in both main() and > > data_loop(). > > I've realized th

Re: [PATCH 0/8] Coverity fixes for vchan-socket-proxy

2020-05-25 Thread Jason Andryuk
On Sun, May 24, 2020 at 10:50 PM Jason Andryuk wrote: > > This series addresses some Coverity reports. To handle closing FDs, a > state struct is introduced to track FDs closed in both main() and > data_loop(). I've realized the changes here are insufficient to handle the FD leaks. That is, the

[PATCH 0/8] Coverity fixes for vchan-socket-proxy

2020-05-24 Thread Jason Andryuk
This series addresses some Coverity reports. To handle closing FDs, a state struct is introduced to track FDs closed in both main() and data_loop(). Jason Andryuk (8): vchan-socket-proxy: Ensure UNIX path NUL terminated vchan-socket-proxy: Check xs_watch return value vchan-socket-proxy: Uni