[SDL_Tizen] fix svace issues. 68/88668/2
authorsunghyun kim <scholb.kim@samsung.com>
Tue, 20 Sep 2016 07:24:20 +0000 (16:24 +0900)
committersunghyun kim <scholb.kim@samsung.com>
Tue, 20 Sep 2016 07:46:35 +0000 (16:46 +0900)
fix some svace issue

Change-Id: I288d8af64bd07f24c3e7ffe4baf9ba9aedfb1d5e

src/test/SDL_test_log.c
src/video/tizen/SDL_tizenmouse.c
src/video/wayland/SDL_waylandmouse.c

index 19f57a0..7156659 100755 (executable)
@@ -87,16 +87,7 @@ void SDLTest_Log(SDL_PRINTF_FORMAT_STRING const char *fmt, ...)
     }
 
     size_t len = SDL_strlen(logMessage);
-    strcpy(message, logMessage);
-
-    // Chop off final endline.
-
-    if ((len > 0) && (message[len-1] == '\n')) {
-        message[--len] = '\0';
-        if ((len > 0) && (message[len-1] == '\r')) {  // catch "\r\n", too. 
-            message[--len] = '\0';
-        }
-    }
+    SDL_strlcpy(message, logMessage,len+1);
 
     SDL_RWops *rwops = SDL_RWFromFile("log.txt", "a+");
     char *text;
@@ -137,16 +128,7 @@ void SDLTest_LogError(SDL_PRINTF_FORMAT_STRING const char *fmt, ...)
     }
 
     size_t len = SDL_strlen(logMessage);
-    strcpy(message, logMessage);
-
-    // Chop off final endline.
-
-    if ((len > 0) && (message[len-1] == '\n')) {
-        message[--len] = '\0';
-        if ((len > 0) && (message[len-1] == '\r')) {  // catch "\r\n", too. 
-            message[--len] = '\0';
-        }
-    }
+    SDL_strlcpy(message, logMessage,len+1);
 
     SDL_RWops *rwops = SDL_RWFromFile("log.txt", "a+");
     char *text;
index 84bfaf5..6aa46ad 100755 (executable)
@@ -112,6 +112,7 @@ create_buffer_from_shm(Tizen_CursorData *d, int width, int height, uint32_t form
         d->shm_data = NULL;
         fprintf (stderr, "mmap () failed\n");
         close (shm_fd);
+        return -1;
     }
 
     shm_pool = wl_shm_create_pool(ecore_wl_shm_get(), shm_fd, size);
index b810f77..38b0d31 100644 (file)
@@ -130,6 +130,7 @@ create_buffer_from_shm(Wayland_CursorData *d,
         d->shm_data = NULL;
         fprintf (stderr, "mmap () failed\n");
         close (shm_fd);
+        return -1;
     }
 
     shm_pool = wl_shm_create_pool(data->shm, shm_fd, size);