Re: [PATCH 2/2] crypto/rijndael: adjust for Misra C:2012 rule 8.3

[Stefano Stabellini <sstabellini@xxxxxxxxxx>]

On Wed, 13 May 2026, Jan Beulich wrote:
> ... ("All declarations of an object or function shall use the same names
> and type qualifiers"). Bring declarations in line with the corresponding
> definitions, accepting the use of the being-phased-out u<N> types. While
> we don't define NEED_RIJNDAEL_DECRYPT, cover rijndaelKeySetupDec() as
> well for completeness.
> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> --- a/xen/include/crypto/rijndael.h
> +++ b/xen/include/crypto/rijndael.h
> @@ -50,9 +50,8 @@ int rijndael_set_key_enc_only(rijndael_c
>  void rijndael_decrypt(rijndael_ctx *ctx, const unsigned char *src, unsigned 
> char *dst);
>  void rijndael_encrypt(rijndael_ctx *ctx, const unsigned char *src, unsigned 
> char *dst);
>  
> -int  rijndaelKeySetupEnc(unsigned int rk[], const unsigned char cipherKey[], 
> int keyBits);
> -int  rijndaelKeySetupDec(unsigned int rk[], const unsigned char cipkerKey[], 
> int keyBits);
> -void rijndaelEncrypt(const unsigned int rk[], int Nr, const unsigned char 
> pt[16],
> -                     unsigned char ct[16]);
> +int  rijndaelKeySetupEnc(u32 rk[], const u8 cipherKey[], int keyBits);
> +int  rijndaelKeySetupDec(u32 rk[], const u8 cipkerKey[], int keyBits);

There is a typo "cipkerKey" which is only present in rijndael.h but not
in rijndael.c


> +void rijndaelEncrypt(const u32 rk[], int Nr, const u8 pt[16], u8 ct[16]);
>  
>  #endif /* __RIJNDAEL_H */
> 
>