X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=secnet.git;a=blobdiff_plain;f=conffile.fl;h=7228c9e8fa6cd672ea5a3ac480d8328fc161cede;hp=c169b572e8a54beae15d7d368f9d5dbc519de78f;hb=45a1170652384baf744a5e61d576af534c7b8b14;hpb=b8dc47dd7590c3b160c1e6b1bdc991f5820e92ad

diff --git a/conffile.fl b/conffile.fl
index c169b57..7228c9e 100644
--- a/conffile.fl
+++ b/conffile.fl
@@ -3,8 +3,11 @@
 
 %option nounput
 %option noinput
+%option never-interactive
 
 %{
+#include <assert.h>
+#include <limits.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
@@ -26,13 +29,13 @@ do{									\
 #define MAX_INCLUDE_DEPTH 10
 struct include_stack_item {
 	YY_BUFFER_STATE bst;
-	uint32_t lineno;
+	int lineno;
 	cstring_t file;
 };
 struct include_stack_item include_stack[MAX_INCLUDE_DEPTH];
 int include_stack_ptr=0;
 
-uint32_t config_lineno=0;
+int config_lineno=0;
 cstring_t config_file="xxx";
 
 static struct p_node *leafnode(uint32_t type)
@@ -68,8 +71,25 @@ static struct p_node *stringnode(string_t string)
 static struct p_node *numnode(string_t number)
 {
 	struct p_node *r;
+	unsigned long n;
 	r=leafnode(T_NUMBER);
-	r->data.number=atoi(number);
+	errno = 0;
+	n = strtoul(number, NULL, 10);
+	/* The caller is expected to only give us [0-9]+,
+	 * so we skip some of the usual syntax checking. */
+	r->data.number=n;
+	/* Give a consistent error message for any kind of
+	 * out-of-range condition */
+	if(errno == ERANGE || n != r->data.number) {
+	    Message(M_FATAL,"config file %s line %d: '%s' is too big\n",
+		    config_file, config_lineno, number);
+	    exit(1);
+	}
+	if(errno) {
+	    Message(M_FATAL,"config file %s line %d: '%s': %s\n",
+		    config_file, config_lineno, number, strerror(errno));
+	    exit(1);
+	}
 	return r;
 }
 
@@ -99,6 +119,7 @@ include			BEGIN(incl);
 	Message(M_FATAL,"config file %s line %d: %s\n",config_file,
 		config_lineno,"``include'' requires a filename");
 	BEGIN(INITIAL);
+	assert(config_lineno < INT_MAX);
 	++config_lineno;
 	++yynerrs;
 }