diff options
| author | Eugene Sandulenko | 2011-04-17 14:48:27 +0300 |
|---|---|---|
| committer | Eugene Sandulenko | 2011-04-17 15:25:06 +0300 |
| commit | ce0cc0ee857740cb0bd1ebd3b4bfa6eefc31aee6 (patch) | |
| tree | 3d4694ec262c99c9b9f3b2dfc00205b323a18ded /backends/platform | |
| parent | a383fe452f5b62a74fc92b8734425bd8ab793402 (diff) | |
| download | scummvm-rg350-ce0cc0ee857740cb0bd1ebd3b4bfa6eefc31aee6.tar.gz scummvm-rg350-ce0cc0ee857740cb0bd1ebd3b4bfa6eefc31aee6.tar.bz2 scummvm-rg350-ce0cc0ee857740cb0bd1ebd3b4bfa6eefc31aee6.zip | |
COMMON: Partial fix for #3087922: COMMON/BASE/BACKENDS: Code analysis warnings
Diffstat (limited to 'backends/platform')
| -rw-r--r-- | backends/platform/sdl/sdl.cpp | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/backends/platform/sdl/sdl.cpp b/backends/platform/sdl/sdl.cpp index d6e79248f9..2c0c45907f 100644 --- a/backends/platform/sdl/sdl.cpp +++ b/backends/platform/sdl/sdl.cpp @@ -378,7 +378,11 @@ void OSystem_SDL::setupIcon() { unsigned int rgba[256]; unsigned int *icon; - sscanf(scummvm_icon[0], "%d %d %d %d", &w, &h, &ncols, &nbytes); + if (sscanf(scummvm_icon[0], "%d %d %d %d", &w, &h, &ncols, &nbytes) != 4) { + warning("Wrong format of scummvm_icon[0] (%s)", scummvm_icon[0]); + + return; + } if ((w > 512) || (h > 512) || (ncols > 255) || (nbytes > 1)) { warning("Could not load the built-in icon (%d %d %d %d)", w, h, ncols, nbytes); return; @@ -393,13 +397,17 @@ void OSystem_SDL::setupIcon() { unsigned char code; char color[32]; unsigned int col; - sscanf(scummvm_icon[1 + i], "%c c %s", &code, color); + if (sscanf(scummvm_icon[1 + i], "%c c %s", &code, color) != 2) { + warning("Wrong format of scummvm_icon[%d] (%s)", 1 + i, scummvm_icon[1 + i]); + } if (!strcmp(color, "None")) col = 0x00000000; else if (!strcmp(color, "black")) col = 0xFF000000; else if (color[0] == '#') { - sscanf(color + 1, "%06x", &col); + if (sscanf(color + 1, "%06x", &col) != 1) { + warning("Wrong format of color (%s)", color + 1); + } col |= 0xFF000000; } else { warning("Could not load the built-in icon (%d %s - %s) ", code, color, scummvm_icon[1 + i]); |