[trivial PATCH 12/22] [otp_mppe.c] eliminate build warnings

Frank Cusack fcusack at fcusack.com
Mon Feb 1 20:20:38 CET 2010


That's incorrect.  The type must be unsigned char.

I didn't look at any other changes, maybe others have problems also.

On February 2, 2010 1:28:45 AM +0800 Wang Tinggong <wangtinggong at gmail.com> 
wrote:
> Signed-off-by: Wang Tinggong <wangtinggong at gmail.com>
> ---
>  src/modules/rlm_otp/otp_mppe.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/modules/rlm_otp/otp_mppe.c
> b/src/modules/rlm_otp/otp_mppe.c index f31ef9f..c5a12c8 100644
> --- a/src/modules/rlm_otp/otp_mppe.c
> +++ b/src/modules/rlm_otp/otp_mppe.c
> @@ -351,7 +351,7 @@ otp_mppe(REQUEST *request, otp_pwe_t pwe, const
> otp_option_t *opt,                                      (2 *
> sizeof(mppe_key)) + 1];  #else /* 0 */
>        /*                           0x   (   ASCII(mppe_key)   )  \0 */
> -      unsigned char mppe_key_string[2 + (2 * sizeof(MasterKey)) + 1];
> +      char mppe_key_string[2 + (2 * sizeof(MasterKey)) + 1];
>  #endif /* else !0 */
>
>        /* Generate the master session key. */







More information about the Freeradius-Devel mailing list