On 8/7/17 1:28 PM, David Ahern wrote: > @@ -99,7 +95,12 @@ static int nl_dump_ext_err(const struct nlmsghdr *nlh, > nl_ext_ack_fn_t errfn) > err_nlh = &err->msg; > } > > - return errfn(errmsg, off, err_nlh); > + if (errfn) > + return errfn(errmsg, off, err_nlh); > + > + fprintf(stderr, "Error: %s\n", errmsg); > + > + return 1;
Dang it, missing an 'if (errmsg)' since it does not have to exist. Will send a v2
