Fix for subtle bug in parsing export area, found by Vaughn Spurlin using Coverity

diff --git a/src/main.cpp b/src/main.cpp
index 9f7bc9ad33d7d9bb9f00d329454eb980a89cf6e3..78b66d8478766b185c65473e63c70a39e01e919c 100644 (file)
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -1345,7 +1345,7 @@ sp_do_export_png(SPDocument *doc)
     if (sp_export_area) {
         /* Try to parse area (given in SVG pixels) */
         gdouble x0,y0,x1,y1;
-        if (!sscanf(sp_export_area, "%lg:%lg:%lg:%lg", &x0, &y0, &x1, &y1) == 4) {
+        if (sscanf(sp_export_area, "%lg:%lg:%lg:%lg", &x0, &y0, &x1, &y1) != 4) {
             g_warning("Cannot parse export area '%s'; use 'x0:y0:x1:y1'. Nothing exported.", sp_export_area);
             return;
         }