Page MenuHomeFreeBSD

crypto: avoid warnings about too-long initializer strings
Needs ReviewPublic

Authored by dim on Fri, Dec 26, 12:14 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 27, 2:51 PM
Unknown Object (File)
Sat, Dec 27, 10:45 AM
Unknown Object (File)
Fri, Dec 26, 11:09 PM
Unknown Object (File)
Fri, Dec 26, 8:16 AM
Unknown Object (File)
Fri, Dec 26, 2:26 AM
Subscribers

Details

Reviewers
cperciva
jhb
markj
Summary

Mark sigma and tau as __non_string, to avoid warnings from
clang 21 similar to:

sys/crypto/chacha20/chacha.c:53:31: error: initializer-string for character array is too long, array size is 16 but initializer has size 17 (including the null terminating character); did you mean to use the 'nonstring' attribute? [-Werror,-Wunterminated-string-initialization]
   53 | static const char sigma[16] = "expand 32-byte k";
      |                               ^~~~~~~~~~~~~~~~~~
sys/crypto/chacha20/chacha.c:54:29: error: initializer-string for character array is too long, array size is 16 but initializer has size 17 (including the null terminating character); did you mean to use the 'nonstring' attribute? [-Werror,-Wunterminated-string-initialization]
   54 | static const char tau[16] = "expand 16-byte k";
      |                             ^~~~~~~~~~~~~~~~~~

MFC after: 3 days

Diff Detail

Repository
rG FreeBSD src repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 69470
Build 66353: arc lint + arc unit