epp: avoid to rely on undefined behavior for fopen.
authorCedric Bail <cedric.bail@samsung.com>
Tue, 5 Nov 2013 07:28:56 +0000 (16:28 +0900)
committerCedric Bail <cedric.bail@samsung.com>
Tue, 5 Nov 2013 07:28:56 +0000 (16:28 +0900)
In the case we didn't define the file to save to, fopen would have
been called with a NULL parameter. The result of that call is not
really defined and could have led to some crash at some point.

src/bin/edje/epp/cpplib.c

index af65241..4314b15 100644 (file)
@@ -7448,8 +7448,9 @@ extern cpp_options         options;
 void
 using_file(const char *filename, const char type)
 {
-   FILE *f;
+   FILE *f = NULL;
 
+   if (!options.watchfile) return;
    f = fopen(options.watchfile, "a");
    if (!f) return;
    if (anotate)