[OpenWrt-Devel] [PATCH libubox 9/9] avl: guard against theoretical null pointer dereference

[Petr Štetiar]

Yousong Zhou <yszhou4tech at gmail.com> [2019-11-20 22:01:22]:

> By the way, will assert(node-parent != NULL) suffice to inform the
> analyzer the underlying details?  If it does, we could also apply it
> to b64_encode(), b64_decode().

BTW in libnl-tiny I've learned, that using assert() in CMake release builds
needs fiddling with C flags as CMake adds -DNDEBUG to release build C flags
which in turn disable assert().

Anyway, I agree, that crashing with invalid inputs is the proper way, so I've
reworked it with assert() as it works for analyzer as well.

-- ynezz

_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel