Changeset 18850 for trunk/ippTools/src/pxadmin.c

Timestamp:

Aug 1, 2008, 9:05:29 AM (18 years ago)

Author:

Paul Price

Message:

Need to check return value of fgets to avoid compiler warnings on some
systems.

File:

• trunk/ippTools/src/pxadmin.c (modified) (2 diffs)

trunk/ippTools/src/pxadmin.c

@@ -128 +128 @@
         psString dbName = psMetadataLookupStr(&status, config->modules->complete, "DBNAME");
 
-        fprintf (stdout, "*** delete the tables from database %s? ***\n", dbName);
-        fprintf (stdout, "*** to delete the tables, answer YES, and give password ***\n");
-        fprintf (stdout, "*** WARNING: this action is permanent ***\n\n");
-
-        fprintf (stdout, "delete the tables (YES/[n]): ");
-        fgets (line, 128, stdin);
-        sscanf (line, "%s", answer);
+        fprintf(stdout, "*** delete the tables from database %s? ***\n", dbName);
+        fprintf(stdout, "*** to delete the tables, answer YES, and give password ***\n");
+        fprintf(stdout, "*** WARNING: this action is permanent ***\n\n");
+
+        fprintf(stdout, "delete the tables (YES/[n]): ");
+        if (!fgets(line, 128, stdin)) {
+            psError(PS_ERR_IO, true, "Unable to read response.");
+            return false;
+        }
+        sscanf(line, "%s", answer);
         if (strcmp (answer, "YES"))  {
             psError(PS_ERR_UNKNOWN, true, "tables NOT deleleted");
@@ -140 +143 @@
         }
 
-        fprintf (stdout, "enter dbh connection password: ");
-        fgets (line, 128, stdin);
-        sscanf (line, "%s", answer);
+        fprintf(stdout, "enter dbh connection password: ");
+        if (!fgets(line, 128, stdin)) {
+            psError(PS_ERR_IO, true, "Unable to read response.");
+            return false;
+        }
+        sscanf(line, "%s", answer);
 
         psString dbPassword = psMetadataLookupStr(&status, config->modules->complete, "DBPASSWORD");