Fixed a potential NULL pointer dereference in exif_loader_get_buf.

This would only happen if given a NULL pointer by the application.
Found by Clang scan-build.

diff --git a/libexif/exif-loader.c b/libexif/exif-loader.c
index 317b86b..e2208b4 100644 (file)
--- a/libexif/exif-loader.c
+++ b/libexif/exif-loader.c
@@ -410,9 +410,12 @@ exif_loader_get_buf (ExifLoader *loader, const unsigned char **buf,
        const unsigned char* b = NULL;
        unsigned int s = 0;
 
-       if (!loader || (loader->data_format == EL_DATA_FORMAT_UNKNOWN)) {
+       if (!loader)
+               return;
+       if (loader->data_format == EL_DATA_FORMAT_UNKNOWN) {
                exif_log (loader->log, EXIF_LOG_CODE_DEBUG, "ExifLoader",
                          "Loader format unknown");
+               return;
        } else {
                b = loader->buf;
                s = loader->bytes_read;