From baba109968b1fb5331e24a543ec009bab62c09f5 Mon Sep 17 00:00:00 2001
From: ian <ian>
Date: Sun, 19 Sep 1999 20:10:54 +0000
Subject: [PATCH] Bugfixes: install service prog in right place; allow no extra
 routes.

---
 ipif/Makefile  |  2 +-
 ipif/service.c | 10 +++++-----
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/ipif/Makefile b/ipif/Makefile
index c5e3a37..24d74ad 100644
--- a/ipif/Makefile
+++ b/ipif/Makefile
@@ -18,6 +18,6 @@ all:		$(TARGETS)
 
 install:	all
 		mkdir -p $(libuserv) $(services)
-		cp service $(libdir)/ipif
+		cp service $(libuserv)/ipif
 		cp ipif $(services)/ipif:new
 		set -e; cd $(services); test -f ipif || mv ipif:new ipif
diff --git a/ipif/service.c b/ipif/service.c
index 5ed6014..ad55e30 100644
--- a/ipif/service.c
+++ b/ipif/service.c
@@ -202,13 +202,13 @@ static unsigned long eat_number(const char **argp, const char *what,
   char *ep;
   int endchar;
 
-  if (!*argp) { badusage("missing number %s\n",what); }
+  if (!*argp) { badusage("missing number %s",what); }
   rv= strtoul(*argp,&ep,0);
   if ((endchar= *ep)) {
-    if (!endchars) badusage("junk after number %s\n",what);
+    if (!endchars) badusage("junk after number %s",what);
     if (!strchr(endchars,endchar))
       badusage("invalid character or delimiter `%c' in or after number, %s:"
-	       " expected %s (or none?)\n", endchar,what,endchars);
+	       " expected %s (or none?)", endchar,what,endchars);
     *argp= ep+1;
   } else {
     *argp= 0;
@@ -259,7 +259,7 @@ static void eat_prefixmask(const char **argp, const char *what,
   len= eat_number(argp,whatbuf, 0,32, endchars,endchar_r);
 
   mask= (~0UL << (32-len));
-  if (prefix & ~mask) badusage("%s prefix %08lx not fully contained in mask %08lx\n",
+  if (prefix & ~mask) badusage("%s prefix %08lx not fully contained in mask %08lx",
 			       what,prefix,mask);
   *prefix_r= prefix;
   if (mask_r) *mask_r= mask;
@@ -432,7 +432,7 @@ static void parseargs(int argc, const char *const *argv) {
   
   carg= *++argv;
   for (nexroutes=0;
-       carg;
+       carg && *carg;
        nexroutes++) {
     if (nexroutes == MAXEXROUTES)
       fatal("too many extra routes (only %d allowed)",MAXEXROUTES);
-- 
2.30.2