diff options
author | Stefan Schmidt <stefan@osg.samsung.com> | 2016-12-05 16:30:22 +0100 |
---|---|---|
committer | Stefan Schmidt <stefan@osg.samsung.com> | 2016-12-05 17:07:39 +0100 |
commit | 6cf11650c451a93fd6fbd8c3424d2f724791109d (patch) | |
tree | e3f135e2d1ca9ac4d7620d5d9280a9714427b484 | |
parent | e1fd028081507c87c5b6245881f8408eca1e572c (diff) | |
download | efl-6cf11650c451a93fd6fbd8c3424d2f724791109d.tar.gz |
edje: svg_loader: make sure we check the return from all _parser_number calls
Instead of nesting this make a bit more sense (at least to me). Also return in
case one of these is not a number.
CID: 1356610
-rw-r--r-- | src/bin/edje/edje_svg_loader.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/src/bin/edje/edje_svg_loader.c b/src/bin/edje/edje_svg_loader.c index 781f141c25..c9b4165cbe 100644 --- a/src/bin/edje/edje_svg_loader.c +++ b/src/bin/edje/edje_svg_loader.c @@ -680,10 +680,12 @@ _attr_parse_svg_node(void *data, const char *key, const char *value) } else if (!strcmp(key, "viewBox")) { - if (_parse_number(&value, &doc->vx)) - if (_parse_number(&value, &doc->vy)) - if (_parse_number(&value, &doc->vw)) - _parse_number(&value, &doc->vh); + + if (!_parse_number(&value, &doc->vx) && !_parse_number(&value, &doc->vy) && + !_parse_number(&value, &doc->vw) && !_parse_number(&value, &doc->vh)) + { + return EINA_FALSE; + } } else if (!strcmp(key, "style")) { |