Make sure the integer passed to --result-fd is correct before manipulating

the pointers(we checked after we manipulated previously)

diff --git a/ChangeLog b/ChangeLog
index d7c61d3034fbc1db78738b5da76974c9c9c9332e..4f417f1f9082a56ac73f59f33de583103115e397 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+Mon Oct  8 19:48:28 CDT 2001 Adam Heath <doogie@debian.org>
+
+  * main/main.c: Make sure the integer passed to --result-fd is correct
+    before manipulating the pointers(we checked after we manipulated
+    previously)
+
 Sun Oct  7 22:23:54 CDT 2001 Adam Heath <doogie@debian.org>
 
   * Makefile.in: Remove dulicate inclusion of utils in SUBDIRS.

diff --git a/main/main.c b/main/main.c
index a4c8b4e824b3c5eddb3632295bd0417372227bf4..872d80066754b0e25195fd9888c13fbd3f25fd5f 100644 (file)
--- a/main/main.c
+++ b/main/main.c
@@ -270,15 +270,15 @@ static void setstatuspipe(const struct cmdinfo *cip, const char *value) {
   unsigned long v;
   char *ep;
 
+  v= strtoul(value,&ep,0);
+  if (*ep || v > INT_MAX)
+    badusage(_("invalid integer for --%s: `%.250s'"),cip->olong,value);
   if (lastpipe) {
     lastpipe->next= nfmalloc(sizeof(struct pipef));
     lastpipe= lastpipe->next;
   } else {
     lastpipe= status_pipes= nfmalloc(sizeof(struct pipef));
   }
-  v= strtoul(value,&ep,0);
-  if (*ep || v > INT_MAX)
-    badusage(_("invalid integer for --%s: `%.250s'"),cip->olong,value);
   lastpipe->fd= v;
 }