fixed wrong patch with the return value of fgets into convert.c file

2 files changed, 5 insertions, 4 deletions

diff --git a/CHANGES b/CHANGES
index 1a645c4e..c9f3b724 100644
--- a/CHANGES
+++ b/CHANGES
@@ -5,6 +5,9 @@ What's New for OpenJPEG
 ! : changed
 + : added
 
+August 11, 2011
+! [mickael] fixed wrong patch with the return value of fgets into convert.c file
+
 August 10, 2011
 ! [mickael] removed unused parameters warnings with the solution proposed by myself and Bob Friesenhahn.
 ! [mickael] add copyright header into new test functions.
diff --git a/applications/codec/convert.c b/applications/codec/convert.c
index 18fa03ff..c7afc074 100644
--- a/applications/codec/convert.c
+++ b/applications/codec/convert.c
@@ -1408,11 +1408,9 @@ static void read_pnm_header(FILE *reader, struct pnm_header *ph)
     char idf[256], type[256];
     char line[256];
 
-    char* return_value_fgets = fgets(line, 250, reader);
-    if (!strcmp(return_value_fgets,line))
+    if (fgets(line, 250, reader) == NULL)
     {
-    	fprintf(stderr,"\nWARNING: fgets return a value different that the first argument");
-    	free(return_value_fgets);
+    	fprintf(stderr,"\nWARNING: fgets return a NULL value");
     	return;
     }