[ovs-dev] [PATCH] netdev-dpdk: Avoid undefined behavior processing devargs
blp at ovn.org
Mon Jan 13 17:34:45 UTC 2020
On Thu, Jan 09, 2020 at 01:27:06PM -0500, Aaron Conole wrote:
> In "Use of library functions" in the C standard, the following statement
> is written to apply to all library functions:
> If an argument to a function has an invalid value (such as ... a
> null pointer ... the behavior is undefined.
> Later, under the "String handling" section, "Comparison functions" no
> exception is listed for strcmp, which means NULL is invalid. It may
> be possible for the smap_get to return NULL.
It's not just a technicality for strcmp(), this actually causes SIGSEGV
with most libc, including glibc.
Anyway, I'm glad that this got fixed.
More information about the dev