add error log in lxc_file_for_each_line

this is required to log error where needed and to to avoid additionnal log later on. Signed-off-by: Michel Normand <normand@fr.ibm.com> Signed-off-by: Daniel Lezcano <dlezcano@fr.ibm.com>

add error log in lxc_file_for_each_line
2e714984 · Michel Normand · Daniel Lezcano · 487f4ee9 · 2e714984 · 2e714984
Commit 2e714984 authored May 14, 2009 by Michel Normand Committed by Daniel Lezcano May 14, 2009
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 3 deletions

parse.c src/lxc/parse.c +4 -2

parse.h src/lxc/parse.h +1 -1

No files found.
--- a/src/lxc/parse.c
+++ b/src/lxc/parse.c
@@ -64,7 +64,7 @@ int lxc_dir_for_each(const char *name, const char *directory,
 }

 int lxc_file_for_each_line(const char *file, lxc_file_cb callback,
-			   void *buffer, size_t len, void* data)
+			   char *buffer, size_t len, void* data)
 {
 	FILE *f;
 	int err = -1;
@@ -77,8 +77,10 @@ int lxc_file_for_each_line(const char *file, lxc_file_cb callback,

 	while (fgets(buffer, len, f)) {
 		err = callback(buffer, data);
-		if (err)
+		if (err) {
+			ERROR("failed to process '%s'", buffer);
 			goto out;
+		}
 	}
 out:
 	fclose(f);

--- a/src/lxc/parse.h
+++ b/src/lxc/parse.h
@@ -32,7 +32,7 @@ extern int lxc_dir_for_each(const char *name, const char *directory,
 			    lxc_dir_cb callback, void *data);

 extern int lxc_file_for_each_line(const char *file, lxc_file_cb callback,
-				  void *buffer, size_t len, void* data);
+				  char *buffer, size_t len, void* data);

 extern int lxc_char_left_gc(char *buffer, size_t len);