summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCarsten Haitzler (Rasterman) <raster@rasterman.com>2017-07-28 20:39:44 +0900
committerCarsten Haitzler (Rasterman) <raster@rasterman.com>2017-07-29 09:08:40 +0900
commit64dc6ab056bed36e21c215c47fb81f26dec3396e (patch)
tree3000841b57f524e63fb11c0cbeccca9bc82d37b2
parent02a07caca4c33c76c8174d6bc8e2edbd9e8c4a0d (diff)
evas bmp loader - fix warnings about same type handled twice
so a type we handle earlir inan if we re-handle as invalid later. this wouldnt lead to a crash or bugs as the if's would ned to be evaluated in order normally, but it's good to get it right. found by PVS studio
-rw-r--r--src/modules/evas/image_loaders/bmp/evas_image_load_bmp.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/modules/evas/image_loaders/bmp/evas_image_load_bmp.c b/src/modules/evas/image_loaders/bmp/evas_image_load_bmp.c
index ccf983e2db..c14cc4be3f 100644
--- a/src/modules/evas/image_loaders/bmp/evas_image_load_bmp.c
+++ b/src/modules/evas/image_loaders/bmp/evas_image_load_bmp.c
@@ -436,7 +436,7 @@ evas_image_load_file_head_bmp(void *loader_data,
436 } 436 }
437 else if (header.comp == 4) // jpeg - only printer drivers 437 else if (header.comp == 4) // jpeg - only printer drivers
438 goto close_file; 438 goto close_file;
439 else if (header.comp == 3) // png - only printer drivers 439 else if (header.comp == 5) // png - only printer drivers
440 goto close_file; 440 goto close_file;
441 else 441 else
442 goto close_file; 442 goto close_file;
@@ -1405,7 +1405,7 @@ evas_image_load_file_data_bmp(void *loader_data,
1405 { 1405 {
1406 goto close_file; 1406 goto close_file;
1407 } 1407 }
1408 else if (header.comp == 3) // png - only printer drivers 1408 else if (header.comp == 5) // png - only printer drivers
1409 { 1409 {
1410 goto close_file; 1410 goto close_file;
1411 } 1411 }