Relay-Version: version B 2.10 5/3/83; site utzoo.UUCP
Posting-Version: version B 2.10.2 9/18/84; site ucla-cs.ARPA
Path: utzoo!watmath!clyde!burl!ulysses!allegra!mit-eddie!genrad!panda!talcott!harvard!seismo!rochester!rocksvax!parcvax!hplabs!sdcrdcf!trwrb!trwrba!cepu!ucla-cs!ekrell
From: ekrell@ucla-cs.UUCP
Newsgroups: net.news.b,net.sources.bugs,net.bugs
Subject: bug in pathalias (and fix)
Message-ID: <4045@ucla-cs.ARPA>
Date: Fri, 22-Feb-85 19:38:21 EST
Article-I.D.: ucla-cs.4045
Posted: Fri Feb 22 19:38:21 1985
Date-Received: Wed, 27-Feb-85 21:07:15 EST
Organization: UCLA Computer Science Department
Lines: 30
Xref: watmath net.news.b:1082 net.sources.bugs:241 net.bugs:555

I found this problem when I introduced the csnet sites to the database
used by pathalias: the address for waterloo was stored as
"%s%%waterloo.csnet@csnet-relay.arpa" which works ok since the %% gets
translated into one % by sprintf(). The problem was in the address for
utzoo which was "utzoo!%s%waterloo.csnet@csnet-relay.arpa" (note only one
% here). When this was used by sprintf(), the % simply dissapeared.
  The problem was in pathprintf() in mapit.c. Here's a context diff bug fix:

*** mapit.c.old	Fri Feb 22 16:26:50 1985
--- mapit.c	Fri Feb 22 16:28:06 1985
***************
*** 370,375
  				break;
  			case '%':
  				*++buf = *++path;
  				break;
  			}
  		} else

--- 370,376 -----
  				break;
  			case '%':
  				*++buf = *++path;
+ 				buf++;
  				break;
  			}
  		} else
-- 
    Eduardo Krell               UCLA Computer Science Department
    ekrell@ucla-locus.arpa      ..!{sdcrdcf,ihnp4,trwspp,ucbvax}!ucla-cs!ekrell