generic-library/openssl
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix warnings about ignoring fgets return value

Browse Source
This commit is contained in:
Dr. Stephen Henson 2009-10-04 16:42:56 +00:00
parent c21869fb07
commit 0e039aa797
5 changed files with 31 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
apps/ca.c
View File

@ -1263,7 +1263,12 @@ bad:
BIO_printf(bio_err,"\n%d out of %d certificate requests certified, commit? [y/n]",total_done,total); BIO_printf(bio_err,"\n%d out of %d certificate requests certified, commit? [y/n]",total_done,total);
(void)BIO_flush(bio_err); (void)BIO_flush(bio_err);
buf[0][0]='\0'; buf[0][0]='\0';
fgets(buf[0],10,stdin); if (!fgets(buf[0],10,stdin))
{
BIO_printf(bio_err,"CERTIFICATION CANCELED: I/O error\n");
ret=0;
goto err;
}
if ((buf[0][0] != 'y') && (buf[0][0] != 'Y')) if ((buf[0][0] != 'y') && (buf[0][0] != 'Y'))
{ {
BIO_printf(bio_err,"CERTIFICATION CANCELED\n"); BIO_printf(bio_err,"CERTIFICATION CANCELED\n");
@ -2122,7 +2127,12 @@ again2:
BIO_printf(bio_err,"Sign the certificate? [y/n]:"); BIO_printf(bio_err,"Sign the certificate? [y/n]:");
(void)BIO_flush(bio_err); (void)BIO_flush(bio_err);
buf[0]='\0'; buf[0]='\0';
fgets(buf,sizeof(buf)-1,stdin); if (!fgets(buf,sizeof(buf)-1,stdin))
{
BIO_printf(bio_err,"CERTIFICATE WILL NOT BE CERTIFIED: I/O error\n");
ok=0;
goto err;
}
if (!((buf[0] == 'y') || (buf[0] == 'Y'))) if (!((buf[0] == 'y') || (buf[0] == 'Y')))
{ {
BIO_printf(bio_err,"CERTIFICATE WILL NOT BE CERTIFIED\n"); BIO_printf(bio_err,"CERTIFICATE WILL NOT BE CERTIFIED\n");

7
apps/enc.c
View File

@ -243,7 +243,12 @@ int MAIN(int argc, char **argv)
goto bad; goto bad;
} }
buf[0]='\0'; buf[0]='\0';
fgets(buf,sizeof buf,infile); if (!fgets(buf,sizeof buf,infile))
{
BIO_printf(bio_err,"unable to read key from '%s'\n",
file);
goto bad;
}
fclose(infile); fclose(infile);
i=strlen(buf); i=strlen(buf);
if ((i > 0) && if ((i > 0) &&

3
apps/openssl.c
View File

@ -330,7 +330,8 @@ int main(int Argc, char *Argv[])
else prompt="OpenSSL> "; else prompt="OpenSSL> ";
fputs(prompt,stdout); fputs(prompt,stdout);
fflush(stdout); fflush(stdout);
fgets(p,n,stdin); if (!fgets(p,n,stdin))
goto end;
if (p[0] == '\0') goto end; if (p[0] == '\0') goto end;
i=strlen(p); i=strlen(p);
if (i <= 1) break; if (i <= 1) break;

6
apps/req.c
View File

@ -1441,7 +1441,8 @@ start:
buf[0]='\0'; buf[0]='\0';
if (!batch) if (!batch)
{ {
fgets(buf,sizeof buf,stdin); if (!fgets(buf,sizeof buf,stdin))
return 0;
} }
else else
{ {
@ -1499,7 +1500,8 @@ start:
buf[0]='\0'; buf[0]='\0';
if (!batch) if (!batch)
{ {
fgets(buf,sizeof buf,stdin); if (!fgets(buf,sizeof buf,stdin))
return 0;
} }
else else
{ {

11
crypto/ui/ui_openssl.c
View File

@ -299,7 +299,7 @@ static int is_a_tty;
/* Declare static functions */ /* Declare static functions */
#if !defined(OPENSSL_SYS_WIN16) && !defined(OPENSSL_SYS_WINCE) #if !defined(OPENSSL_SYS_WIN16) && !defined(OPENSSL_SYS_WINCE)
static void read_till_nl(FILE *); static int read_till_nl(FILE *);
static void recsig(int); static void recsig(int);
static void pushsig(void); static void pushsig(void);
static void popsig(void); static void popsig(void);
@ -392,14 +392,16 @@ static int read_string(UI *ui, UI_STRING *uis)
#if !defined(OPENSSL_SYS_WIN16) && !defined(OPENSSL_SYS_WINCE) #if !defined(OPENSSL_SYS_WIN16) && !defined(OPENSSL_SYS_WINCE)
/* Internal functions to read a string without echoing */ /* Internal functions to read a string without echoing */
static void read_till_nl(FILE *in) static int read_till_nl(FILE *in)
{ {
#define SIZE 4 #define SIZE 4
char buf[SIZE+1]; char buf[SIZE+1];
do { do {
fgets(buf,SIZE,in); if (!fgets(buf,SIZE,in))
return 0;
} while (strchr(buf,'\n') == NULL); } while (strchr(buf,'\n') == NULL);
return 1;
} }
static volatile sig_atomic_t intr_signal; static volatile sig_atomic_t intr_signal;
@ -447,7 +449,8 @@ static int read_string_inner(UI *ui, UI_STRING *uis, int echo, int strip_nl)
*p='\0'; *p='\0';
} }
else else
read_till_nl(tty_in); if (!read_till_nl(tty_in))
goto error;
if (UI_set_result(ui, uis, result) >= 0) if (UI_set_result(ui, uis, result) >= 0)
ok=1; ok=1;