[sshfs] [PATCH 4/5] detect lines that are too long

[Mike K. <mi...@pa...>]

---
 sshfs.c |   25 ++++++++++++-------------
 1 files changed, 12 insertions(+), 13 deletions(-)

diff --git a/sshfs.c b/sshfs.c
index 359fbf2..2ba4659 100644
--- a/sshfs.c
+++ b/sshfs.c
@@ -3484,23 +3484,23 @@ static int ssh_connect(void)
 	return 0;
 }
 
-/* remove trailing '\n', like the perl func of the same name */
-static inline void chomp(char *line)
-{
-	char *p = line;
-	if ((p = strrchr(line, '\n')))
-		*p = '\0';
-}
-
 /* number of ':' separated fields in a passwd/group file that we care
  * about */
 #define IDMAP_FIELDS 3
 
 /* given a line from a uidmap or gidmap, parse out the name and id */
 static void parse_idmap_line(char *line, const char* filename,
-		const unsigned int lineno, uint32_t *ret_id, char **ret_name)
+		const unsigned int lineno, uint32_t *ret_id, char **ret_name,
+		const int eof)
 {
-	chomp(line);
+	/* chomp off the trailing newline */
+	char *p = line;
+	if ((p = strrchr(line, '\n')))
+		*p = '\0';
+	else if (!eof) {
+		fprintf(stderr, "%s:%u: line too long\n", filename, lineno);
+		exit(1);
+	}
 	char *tokens[IDMAP_FIELDS];
 	char *tok;
 	int i;
@@ -3518,8 +3518,7 @@ static void parse_idmap_line(char *line, const char* filename,
 		name_tok = tokens[0];
 		id_tok = tokens[2];
 	} else {
-		fprintf(stderr, "Unknown format at line %u of '%s'\n",
-				lineno, filename);
+		fprintf(stderr, "%s:%u: unknown format\n", filename, lineno);
 		exit(1);
 	}
 
@@ -3557,7 +3556,7 @@ static void read_id_map(char *file, uint32_t *(*map_fn)(char *),
 		uint32_t remote_id;
 		char *name;
 
-		parse_idmap_line(line, file, lineno, &remote_id, &name);
+		parse_idmap_line(line, file, lineno, &remote_id, &name, feof(fp));
 
 		uint32_t *local_id = map_fn(name);
 		if (local_id == NULL) {
-- 
1.7.7.3