mirror of
https://github.com/nginx/nginx.git
synced 2026-05-27 11:53:27 -04:00
b0a4d0fb82
Some checks failed
buildbot / buildbot (push) Has been cancelled
In the third call to ngx_pstrdup() for setting cycle->conf_param.data in ngx_init_cycle() we would pass in a nulled ngx_str_t in the case there was no -g command line option passed to nginx. This would result in a memcpy(dst, NULL, 0) which up to and including C23 is Undefined Behaviour. Currently Clang and GCC (in this particular case) just treat this as a no-op, so things just happen to work. However some undefined behaviour sanitizers will throw an error when this is hit, e.g. Clang and the zig compiler and it's probably best not to rely on this behaviour. It's worth noting that the next C standard will make this (and other NULL related operations) defined behaviour. Link: <https://www.open-std.org/jtc1/sc22/wg14/www/docs/n3322.pdf> Closes: https://github.com/nginx/nginx/issues/1079 |
||
|---|---|---|
| .. | ||
| core | ||
| event | ||
| http | ||
| misc | ||
| os | ||
| stream | ||