From c17733b805fa9ad9692ee0115a5e7b5b62570e09 Mon Sep 17 00:00:00 2001
From: "Eric S. Raymond" <esr@thyrsus.com>
Date: Mon, 3 Jan 2000 22:59:49 +0000
Subject: Cleanup patches from Gunther Leber.

svn path=/trunk/; revision=2699
---
 NEWS              |  2 ++
 design-notes.html |  6 +++---
 options.c         | 11 ++++-------
 rcfile_y.y        |  2 +-
 4 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/NEWS b/NEWS
index 934ba533..4ba3a804 100644
--- a/NEWS
+++ b/NEWS
@@ -14,6 +14,8 @@ a Certifying Authority we recognize?).
 * Javier Kohen's replacement for es.po.
 * Munechika SUMIKAWA's patch to make IPv6 version build when POP2 is enabled.
 * Russian translation removed; it was badly garbled.
+* Added Rick van Rein's fetchmaildistrib script to the contrib directory.
+* Gunther Leber's cleanup patches.
 * Note to translators: The bodies of the login-error and timeout-error
   form letters have been changed in driver.c.
 
diff --git a/design-notes.html b/design-notes.html
index 567c5be6..3ba76e62 100644
--- a/design-notes.html
+++ b/design-notes.html
@@ -10,7 +10,7 @@
 <table width="100%" cellpadding=0><tr>
 <td width="30%">Back to <a href="/~esr/index.html">Fetchmail Home Page</a>
 <td width="30%" align=center>To <a href="/~esr/sitemap.html">Site Map</a>
-<td width="30%" align=right>$Date: 1999/12/19 20:15:47 $
+<td width="30%" align=right>$Date: 2000/01/03 22:59:48 $
 </table>
 <HR>
 <H1 ALIGN=CENTER>Design Notes On Fetchmail</H1>
@@ -334,7 +334,7 @@ following minimum steps.
 
 <LI>Add proper <code>FLAG_MERGE</code> actions in fetchmail.c's
     optmerge() function.  (If it's a global option, add an override 
-    at the end of load_params.
+    at the end of load_params.)
 
 <LI>Document the option in fetchmail.man.  This will require at least
     two changes; one to the collected table of options, and one full
@@ -535,7 +535,7 @@ all shaped the design in one way or another.<P>
 <table width="100%" cellpadding=0><tr>
 <td width="30%">Back to <a href="index.html">Fetchmail Home Page</a>
 <td width="30%" align=center>To <a href="/~esr/sitemap.html">Site Map</a>
-<td width="30%" align=right>$Date: 1999/12/19 20:15:47 $
+<td width="30%" align=right>$Date: 2000/01/03 22:59:48 $
 </table>
 
 <P><ADDRESS>Eric S. Raymond <A HREF="mailto:esr@thyrsus.com">&lt;esr@snark.thyrsus.com&gt;</A></ADDRESS>
diff --git a/options.c b/options.c
index 4129342b..f4210f62 100644
--- a/options.c
+++ b/options.c
@@ -299,17 +299,14 @@ struct query *ctl;	/* option record to be initialized */
 	    break;
 	case 'f':
 	case LA_RCFILE:
-	    rcfile = (char *) xmalloc(strlen(optarg)+1);
-	    strcpy(rcfile,optarg);
+	    rcfile = (char *) xstrdup(optarg);
 	    break;
 	case 'i':
 	case LA_IDFILE:
-	    rctl->idfile = (char *) xmalloc(strlen(optarg)+1);
-	    strcpy(rctl->idfile,optarg);
+	    rctl->idfile = (char *) xstrdup(optarg);
 	    break;
 	case LA_POSTMASTER:
-	    rctl->postmaster = (char *) xmalloc(strlen(optarg)+1);
-	    strcpy(rctl->postmaster,optarg);
+	    rctl->postmaster = (char *) xstrdup(optarg);
 	    break;
 	case LA_NOBOUNCE:
 	    run.bouncemail = FALSE;
@@ -476,7 +473,7 @@ struct query *ctl;	/* option record to be initialized */
 	    do {
 		struct idlist	*idp = save_str(&ctl->antispam, NULL, 0);;
 
-		idp->val.status.num = atoi(cp);
+		idp->val.status.num = xatoi(cp, &errflag);
 	    } while
 		((cp = strtok((char *)NULL, ",")));
 	    break;
diff --git a/rcfile_y.y b/rcfile_y.y
index 4cb0b5a4..525a38d5 100644
--- a/rcfile_y.y
+++ b/rcfile_y.y
@@ -388,7 +388,7 @@ int prc_filecheck(const char *pathname, const flag securecheck)
 	}
     }
 
-    if (!securecheck)	return 0;
+    if (!securecheck)	return PS_SUCCESS;
 
     if ((statbuf.st_mode & S_IFLNK) == S_IFLNK)
     {
-- 
cgit v1.2.3