diff options
| author | Quentin Young <qlyoung@qlyoung.net> | 2021-02-07 22:39:42 -0500 |
|---|---|---|
| committer | Quentin Young <qlyoung@nvidia.com> | 2021-02-09 15:40:40 -0500 |
| commit | 7533cad751eaeabe5626cad636aefa8bf50d7409 (patch) | |
| tree | 866b7345bfbef16c3ae473aa997929598b61d45a /doc/developer/workflow.rst | |
| parent | 658186e8b6ad84bd92fa46201901cf75bf3174c2 (diff) | |
*: remove more sprintf()
Should be just a couple non-development, non-test occurrences of this
function left now.
Signed-off-by: Quentin Young <qlyoung@qlyoung.net>
Diffstat (limited to 'doc/developer/workflow.rst')
| -rw-r--r-- | doc/developer/workflow.rst | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/doc/developer/workflow.rst b/doc/developer/workflow.rst index 861d87b998..71e2b00448 100644 --- a/doc/developer/workflow.rst +++ b/doc/developer/workflow.rst @@ -563,7 +563,7 @@ In general, code submitted into FRR will be rejected if it uses unsafe programming practices. While there is no enforced overall ruleset, the following requirements have achieved consensus: -- ``strcpy``, ``strcat`` and ``sprintf`` are inacceptable without exception. +- ``strcpy``, ``strcat`` and ``sprintf`` are unacceptable without exception. Use ``strlcpy``, ``strlcat`` and ``snprintf`` instead. (Rationale: even if you know the operation cannot overflow the buffer, a future code change may inadvertedly introduce an overflow.) |