From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sasha Khapyorsky Subject: Re: [PATCH] infiniband-diags/ibqueryerrors: Fix realloc size to prevent core Date: Wed, 4 Nov 2009 00:22:18 +0200 Message-ID: <20091103222218.GG29388@me> References: <20091103132003.112e10b0.weiny2@llnl.gov> <1257284362.580.59.camel@auk31.llnl.gov> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1257284362.580.59.camel-X2zTWyBD0EhliZ7u+bvwcg@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Al Chu Cc: Ira Weiny , "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: linux-rdma@vger.kernel.org On 13:39 Tue 03 Nov , Al Chu wrote: > Need to check for NULL? Manpage says its possible: > > --- > realloc() returns a pointer to the newly allocated memory, which is > suitably aligned for any kind of variable and may be different from ptr, > or NULL if the request fails. > --- > > Al > > On Tue, 2009-11-03 at 13:20 -0800, Ira Weiny wrote: > > From: Ira Weiny > > Date: Tue, 3 Nov 2009 13:19:33 -0800 > > Subject: [PATCH] infiniband-diags/ibqueryerrors: Fix realloc size to prevent core > > > > Realloc size was wrong causing a core when enough errors were > > suppressed. Reproduced by running: > > > > ibqueryerrors -c -s RcvSwRelayErrors,LinkDowned,VL15Dropped,XmtWait,SymbolErrors,LinkRecovers,RcvErrors > > > > Signed-off-by: Ira Weiny > > --- > > infiniband-diags/src/ibqueryerrors.c | 3 ++- > > 1 files changed, 2 insertions(+), 1 deletions(-) > > > > diff --git a/infiniband-diags/src/ibqueryerrors.c b/infiniband-diags/src/ibqueryerrors.c > > index f83f29e..aac7087 100644 > > --- a/infiniband-diags/src/ibqueryerrors.c > > +++ b/infiniband-diags/src/ibqueryerrors.c > > @@ -381,7 +381,8 @@ void print_node(ibnd_node_t * node, void *user_data) > > > > static void add_suppressed(enum MAD_FIELDS field) > > { > > - suppressed_fields = realloc(suppressed_fields, sizeof(enum MAD_FIELDS)); > > + suppressed_fields = realloc(suppressed_fields, > > + (sup_total+1)*sizeof(enum MAD_FIELDS)); Also it is never freed. Maybe just to have a static array (0-terminated) and to not bother with realloc()? Sasha > > suppressed_fields[sup_total] = field; > > sup_total++; > > } > -- > Albert Chu > chu11-i2BcT+NCU+M@public.gmane.org > Computer Scientist > High Performance Systems Division > Lawrence Livermore National Laboratory > > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html