Indeed, cn could be NULL if we're testing that case... not sure that
code is very useful in this form (nobody ever builds -DSOMETHING_TEST
these days) - it should become a unit test whenever we work on pool
handling next time.
This said, if it can be NULL, and we're not on an OS that tolerates
"pass NULL to %s", this breaks in the wrong place, so fixing is good
(not something compiled on 2.7, so only fixing master).
Your patch has been applied to the master branch.
commit b887872524d6fc481c23bfac20c3f5d117547eba
Author: Frank Lichtenheld
Date: Fri May 8 17:31:51 2026 +0200
pool: Fix a potential NULL pointer de-reference in test code
Signed-off-by: Frank Lichtenheld <[email protected]>
Acked-by: Gert Doering <[email protected]>
Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1676
Message-Id: <[email protected]>
URL:
https://www.mail-archive.com/[email protected]/msg36865.html
Signed-off-by: Gert Doering <[email protected]>
--
kind regards,
Gert Doering
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
