On Mon, Jan 11, 2016 at 01:07:06PM -0500, Russell Bryant wrote:
> On 01/11/2016 12:21 PM, Ben Pfaff wrote:
> > This avoids a null pointer dereference in the case of memory allocation
> > failure.
> >
> > Found by inspection.
> >
> > Signed-off-by: Ben Pfaff <b...@ovn.org>
> > ---
> > lib/ofp-parse.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/lib/ofp-parse.c b/lib/ofp-parse.c
> > index 6fb43f5..5ef3892 100644
> > --- a/lib/ofp-parse.c
> > +++ b/lib/ofp-parse.c
> > @@ -893,7 +893,7 @@ parse_ofp_table_vacancy(struct ofputil_table_mod *tm,
> > const char *setting)
> > {
> > char *save_ptr = NULL;
> > char *vac_up, *vac_down;
> > - char *value = strdup(setting);
> > + char *value = xstrdup(setting);
> > char *ret_msg;
> > int vacancy_up, vacancy_down;
> >
> >
>
> Acked-by: Russell Bryant <russ...@ovn.org>
Thanks, applied to master and branch-2.5.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
