Fixed.  Good catch.

In message <[email protected]> on Tue, 08 Jul 
2014 22:35:37 +0200, [email protected] said:

dcruette> Hello
dcruette> 
dcruette> Seems to be a typo error in crypto/ui/ui_lib.c:919
dcruette> 
dcruette> Present in openssl-SNAP-20140703
dcruette> 
dcruette> Compilation is Ok but switch/case/default structure error
dcruette> 
dcruette> This portion
dcruette>       case UIT_BOOLEAN:
dcruette>               {
dcruette>               const char *p;
dcruette> 
dcruette>               if (!uis->result_buf)
dcruette>                       {
dcruette>                       UIerr(UI_F_UI_SET_RESULT,UI_R_NO_RESULT_BUFFER);
dcruette>                       return -1;
dcruette>                       }
dcruette> 
dcruette>               uis->result_buf[0] = '\0';
dcruette>               for(p = result; *p; p++)
dcruette>                       {
dcruette>                       if (strchr(uis->_.boolean_data.ok_chars, *p))
dcruette>                               {
dcruette>                               uis->result_buf[0] =
dcruette>                                       uis->_.boolean_data.ok_chars[0];
dcruette>                               break;
dcruette>                               }
dcruette>                       if (strchr(uis->_.boolean_data.cancel_chars, 
*p))
dcruette>                               {
dcruette>                               uis->result_buf[0] =
dcruette>                                       
uis->_.boolean_data.cancel_chars[0];
dcruette>                               break;
dcruette>                               }
dcruette>                       }
dcruette>       default:
dcruette>               break;
dcruette>               }
dcruette>               }
dcruette> 
dcruette> Should be
dcruette>       case UIT_BOOLEAN:
dcruette>               {
dcruette>               const char *p;
dcruette> 
dcruette>               if (!uis->result_buf)
dcruette>                       {
dcruette>                       UIerr(UI_F_UI_SET_RESULT,UI_R_NO_RESULT_BUFFER);
dcruette>                       return -1;
dcruette>                       }
dcruette> 
dcruette>               uis->result_buf[0] = '\0';
dcruette>               for(p = result; *p; p++)
dcruette>                       {
dcruette>                       if (strchr(uis->_.boolean_data.ok_chars, *p))
dcruette>                               {
dcruette>                               uis->result_buf[0] =
dcruette>                                       uis->_.boolean_data.ok_chars[0];
dcruette>                               break;
dcruette>                               }
dcruette>                       if (strchr(uis->_.boolean_data.cancel_chars, 
*p))
dcruette>                               {
dcruette>                               uis->result_buf[0] =
dcruette>                                       
uis->_.boolean_data.cancel_chars[0];
dcruette>                               break;
dcruette>                               }
dcruette>                       }
dcruette>               }
dcruette>       default:
dcruette>               break;
dcruette>               }
dcruette> 
dcruette> Didier CRUETTE
dcruette> ______________________________________________________________________
dcruette> OpenSSL Project                                 http://www.openssl.org
dcruette> Development Mailing List                       [email protected]
dcruette> Automated List Manager                           [email protected]
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [email protected]
Automated List Manager                           [email protected]

Reply via email to