Fix improper errno handling 91/195991/1 accepted/tizen/unified/20181221.125511 submit/tizen/20181221.024839
authorSeungbae Shin <seungbae.shin@samsung.com>
Thu, 20 Dec 2018 10:46:32 +0000 (19:46 +0900)
committerSeungbae Shin <seungbae.shin@samsung.com>
Thu, 20 Dec 2018 10:46:32 +0000 (19:46 +0900)
[Version] 0.12.33
[Issue Type] Bug

Change-Id: If13ea55bbce212bf96581a66b9e464019e7b8819

focus_server/mm_sound_mgr_focus_socket.c
mm_sound_bootsound.c
mm_sound_keysound.c
packaging/libmm-sound.spec

index 16a15ff..3532235 100644 (file)
@@ -430,9 +430,10 @@ int MMSoundMgrFocusSocketInit(int *fd)
        strncpy(addr_un.sun_path, FOCUS_SERVER_SOCK, sizeof(addr_un.sun_path));
 
        if (bind(socket_fd, (struct sockaddr *)&addr_un, sizeof(addr_un))) {
-               strerror_r(errno, str_error, sizeof(str_error));
+               int errsv = errno;
+               strerror_r(errsv, str_error, sizeof(str_error));
                debug_error("failed to bind(), err: %s", str_error);
-               if (errno == EADDRINUSE) {
+               if (errsv == EADDRINUSE) {
                        unlink(FOCUS_SERVER_SOCK);
                        debug_msg("unlink socket and bind again...");
                        if (bind(socket_fd, (struct sockaddr *)&addr_un, sizeof(addr_un))) {
index 0924a25..a505231 100644 (file)
@@ -104,9 +104,10 @@ int mm_sound_boot_play_sound(char* path)
        fd = open(path, O_RDONLY);
        if (fd == -1) {
                char str_error[256];
-               strerror_r(errno, str_error, sizeof(str_error));
-               debug_error("file open failed with [%s][%d]", str_error, errno);
-               switch (errno) {
+               int errsv = errno;
+               strerror_r(errsv, str_error, sizeof(str_error));
+               debug_error("file open failed with [%s][%d]", str_error, errsv);
+               switch (errsv) {
                case ENOENT:
                        return MM_ERROR_SOUND_FILE_NOT_FOUND;
                default:
index dcac051..9f42330 100644 (file)
@@ -173,9 +173,10 @@ static int _mm_sound_play_keysound(const char *filename, int volume_config, ipc_
                fd = open(filename, O_RDONLY);
                if (fd == -1) {
                        char str_error[256];
-                       strerror_r(errno, str_error, sizeof(str_error));
-                       debug_error("file open failed with [%s][%d]", str_error, errno);
-                       switch (errno) {
+                       int errsv = errno;
+                       strerror_r(errsv, str_error, sizeof(str_error));
+                       debug_error("file open failed with [%s][%d]", str_error, errsv);
+                       switch (errsv) {
                        case ENOENT:
                                return MM_ERROR_SOUND_FILE_NOT_FOUND;
                        default:
index 8fe95a3..32e868b 100644 (file)
@@ -1,6 +1,6 @@
 Name:       libmm-sound
 Summary:    MMSound Package contains client lib and sound_server binary
-Version:    0.12.32
+Version:    0.12.33
 Release:    0
 Group:      System/Libraries
 License:    Apache-2.0