Re: [lustre-devel] [bug report] staging: lustre: replace simple cases of LIBCFS_ALLOC with kzalloc.

2018-01-16 Thread Dan Carpenter
On Mon, Jan 15, 2018 at 06:27:26PM +, Eremin, Dmitry wrote: > Hello Dan, > > The function kiblnd_destroy_conn() is conditionally free the conn pointer. > > void kiblnd_destroy_conn(kib_conn_t *conn, bool free_conn) > { > […] > if (free_conn) >LI

RE: [lustre-devel] [bug report] staging: lustre: replace simple cases of LIBCFS_ALLOC with kzalloc.

2018-01-15 Thread Eremin, Dmitry
e.org > Subject: RE: [lustre-devel] [bug report] staging: lustre: replace simple cases > of LIBCFS_ALLOC with kzalloc. > > Hello Dan, > > The function kiblnd_destroy_conn() is conditionally free the conn pointer. > > void kiblnd_destroy_conn(kib_conn_t *conn, bool free_conn

RE: [lustre-devel] [bug report] staging: lustre: replace simple cases of LIBCFS_ALLOC with kzalloc.

2018-01-15 Thread Eremin, Dmitry
ubject: [lustre-devel] [bug report] staging: lustre: replace simple cases of > LIBCFS_ALLOC with kzalloc. > > [ This code was already buggy, it's just that Neil's change made it >show up in static analysis. - dan ] > > Hello NeilBrown, > > The patch 3c88b