Remove unnecessary workqueue stuff

Change-Id: If13593ab411f744ae203b6fdbb4e7146feaa9747

Consider as normal operation when destroy is executed before job function of data is triggered

Change-Id: I59a9766e50cbe426d71814522e286aa5514dc19a

Do not create a dispatcher thread when accessing an invalid fd

Change-Id: I82df329239bf6a1965ee770865cbca16522c9f2a

Add workqueue job after confirming that the module is valid

Change-Id: Iec473889224723e137fe4bb5a347cb386a6177be

Avoid the module access after destroy

Change-Id: Id6e5779dc8ef38b4625cd0b8c597ea2a733b2cc9

Add the removed data channel creation

Change-Id: Iedc071d6465b31bf8a438f87eb5234881e0770c2

Fix crash when create dispatch is failed

- The mutex is unlocked again after unlocked properly.

[Version] 0.3.87
[Profile] Common
[Issue Type] Bug fix

Change-Id: I352fb09b18cebefa07a413c0e2f2817c170454c6
Signed-off-by: Jeongmo Yang <jm80.yang@samsung.com>

Create the diag thread after daemonizing

 - remove pid file before entering idle at on-demand
 - update gst parameter log
 - update ms_init and ms_run part
 - move ms prefix function of server.c to server_private.c
 - set server ready after daemonizing

Change-Id: I90c4358ee8f9373f896070d2709df237e7ab03e5

Update the cyclomatic complexity of SAM violation issues

Change-Id: Ia9df682ca22f707276d03ec83636e050fd3d7c15

Update duplicate part to get JSON value

Change-Id: I8a9b16944fdf44fbc705407c9d38c7359256eaaf

Fix the issue of heap-use-after-free

 - Use only the more accurate states than muse running
 - Remove muse running

Change-Id: If19b1beae1a2849876ee1acac3b83384e8132999

Change the level of log about instance number and idle memory to warning

Change-Id: Ia98a9162e3b5668c9745a2a5623fb580a40b3707

Module without data thread does not execute a data socket close

 - update the thread exit

Change-Id: Ieabaab75cf8166d3942694b42147d4259efecef3

fix the issue of memory leak

Change-Id: I8d7bde7265fc3329aca8b20473779ec9d044b93a

Remove unused path files

Change-Id: Ib0ad985a76db8d08df30f1f4954aa29bc6951c62

Merge "Add new api to get the value of on-demand" into tizen

Add new api to get the value of on-demand

Change-Id: I155e929dd268f225ff2d72b5676529c1d51f6ad3

Change the config value of the systemd service ("false" -> "no")

"no" and "false" have the same meaning, but the document has "no".
Also, since the parsing order is "no" first, there is some performance gain.

Change-Id: Idba0756318e6bd5b31fc4249017ca1a454e98294

Update to lock/unlock entirely

Change-Id: I74e12c65238ef676b2cd62ff5a9b05b86cb22b1c

Add mutex to prevent json object corruption of free

Change-Id: I3ac6d5932b1771e898b47fc13411ca7d06c009e2

Substitute to use pthead attribute

Change-Id: Icb29d5fe5ce8a6b93fc6e07be1acdaa698b195aa

Merge "Able to get client pid at muse module simply" into tizen

Set the sleep value of the comment appropriately

Change-Id: I1be6bdc1eb22e1b003bae2399b24a9d04bdde9ea

Able to get client pid at muse module simply

Change-Id: Ie187a6fb04c77599d112a2790d06b8b12df3b563

Remove thread check in _ms_ipc_dispatch_worker function.

- The thread variable could not be assigned at that time,
  although it's created successfully by thread scheduling.
  It can cause unexpected invalid operation.

[Version] 0.3.77
[Profile] Common
[Issue Type] Bug fix

Change-Id: Id860d8c2df21da6270866ce2367e711224bf2c43
Signed-off-by: Jeongmo Yang <jm80.yang@samsung.com>

Fix the heap-use-after-free of workqueue thread

 - Add a mutex at calling the msg thread creation of the workqueue
 - Set null after msg thread unref
 - Change variable name of thread

Change-Id: I5d4a6acb05d1920dc5da7970ee68d5c762b504e4

Add sleep to avoid busy looping on epoll_wait

Change-Id: I80331394be0965978bca17a9fee0d5951d46118f

Avoid dispatching if RM is not launched to prevent unnecessary crash

 - Remove duplicate argument
 - Add the service order related with RM

Change-Id: I94b092ae1e3ca57af3ae518b2045b181a8cb260c

fixup! Execute dispatcher worker only when muse-server is ready

Change-Id: If0103cfd2fe08643bedaca00c8e8af67e56f5766

Execute dispatcher worker only when muse-server is ready

Change-Id: I046a5e3509c70c16e95e5fe4a0cf01429b19e1ae

Remove the json free at module cleanup because already freed

Change-Id: I37565f58a87c1e2b1d9d785d5847515c8181f7cd

Add new API to get fds

- The API is existed to get only one fd.

[Version] 0.3.70
[Profile] Common
[Issue Type] Update
[Dependency module] N/A

Change-Id: Ie4a53d591ada37b4a953ddace4d4478ac8787276
Signed-off-by: Jeongmo Yang <jm80.yang@samsung.com>

Update to sync with tizen_5.0 branch

Change-Id: Ib86f1499aa0b4432076960c9349a0ba13b6bc449

Merge "Update message parsing error handling" into tizen

Update message parsing error handling

 - Add mutex lock to avoid race condition
 - Do not dispatch after module index error case

Change-Id: I85e94353559d780e55d9cb5096f33e4967a2b53b

Merge "Resolve the ASAN issue of heap-use-after-free" into tizen

Resolve the ASAN issue of heap-use-after-free

 - The issue is accessing connection and workqueue during g_main_quit
    1. Set null after free
    2. Update the 'state' variable during g_main_quit
    3. Update muse_server_config part separately

Change-Id: I2d8daa2714e9f463a29c04c3a028e352d4b2b5b2

Merge "Release the connection only if connected properly" into tizen

Correct pkgconfig dependency

Change-Id: I09a5102051783e94f965df492b23c35093e009fc
Signed-off-by: Jeongmo Yang <jm80.yang@samsung.com>

Revert "Set mm-common only at client packages"

This reverts commit a320e1a44215633d8e4456ed00aed5ef23fdce6f.

Below functions return MM_ERROR_* values, so it's proper to include mm_error header file in muse_core.h
- int muse_core_connection_close(int sock_fd);
- int muse_core_set_nonblocking(int fd, bool value);
- int muse_core_set_socket_timeout(int sock_fd, int timeout_sec);

Change-Id: I7bd806cc7cbd226828b44fcb393d4b0fc53fda57
Signed-off-by: Jeongmo Yang <jm80.yang@samsung.com>

Release the connection only if connected properly

Change-Id: Id2c95ec2ee255612622450771795043439991c64

Set mm-common only at client packages

Change-Id: I5c5d47ab2de739047e1f984501264e580f2fb914
(cherry picked from commit 18e25070d71cbd8a1d7435b213d50cb7defe006c)

Ignore the client connection requests prior to server reaedy

Change-Id: I7d11e68248c6f69133db3e7e6d973a0f0a4bd0ea

Add mm-common to pc file for client packages

[Version] 0.3.67
[Profile] Common
[Issue Type] Bug fix
[Dependency module] N/A

Change-Id: I5763bac9d3115daf17193fd7ff445a8ee9cfbe32
Signed-off-by: Jeongmo Yang <jm80.yang@samsung.com>

Remove the duplicate ms_log_get_cmd_info

Change-Id: I37d6c5aaa4d2a9120cfdc9388166ffd349cc6d6b

Change not to export muse module enumeration

Change-Id: I5531512146c695625fb088ab10b24e92e2381c8f

Change to use new feature of external storage

Change-Id: I916b07af824c8939ab98b79424b73f8fcb2e2da2

Disable storage_change_cb at wearable because there is not external storage

Change-Id: I632ea0c5293203c3e7b014f033d2b394aa459924

Fix the svace issue

Change-Id: Id03c37441b6759414c91767706ec3ccceeb7b002

Update the Security vulnerability

 - Check if there is symbolic link using realopen before calling open()
 - Compare to null character in strncmp

Change-Id: I65d90b2184cfeccad7eb97c97ca0b014660435b4

Change the path file of VD AUDIO

Change-Id: Ibcd3067c3cef8d3c4e107ae5f6c7597b5d9870c2

Send create server ack always

Change-Id: I3ece87bc45570a7ab8528022b94fcb0be56b7a00

Resolve the svace issue

 - It is usually checked for these functions (getpwuid, getgrgid)
 - getgrgid makes no guaranteee of thread safety. Use getgrgid_r function instead

Change-Id: If2817073e9b81347c683583977df2f27373952c2

Update to save errno it may have been changed by library call

Change-Id: I131f6b7f3ad2aba8f9bd2e77e0de6eb0de4a50a1

Update the system part according to build feature

Change-Id: Id01bef38aea3ce39bfcc7fc7fa9e891f1c51ce95

Merge "Add BSD 3-clause License" into tizen

Notify corresponding command when mm-resource-managerd shutdown

Change-Id: Ia2d023a90db7ae4c6f164fff54712a59bcf10995

Add BSD 3-clause License

Change-Id: Ie2833d6b1f0ed5955a35b2c08b825d6fedf10edc

Merge "Update the socket unittest with using muse-server by itself" into tizen

Update the socket unittest with using muse-server by itself

Change-Id: I45ba36f51d1b67e4eb925f5c8d5f2f8a325b809e

Merge "Update to remove unix domain socket files" into tizen

Update to remove unix domain socket files

Change-Id: I716e84d60b2904041e66320fa19c179c8fed5e25

Merge "Update to log additional information for debugging the exception errors" into tizen

Update to log additional information for debugging the exception errors

 - Additional information is Unix domain socket using system call and
   the user (uid, user name) and group (gid, group name) using new function (ms_log_user_group_info)

Change-Id: I0e66a61568fed1f39aa3e1230ac341b7c9ab3b13

Fix the dlog's invalid format

Change-Id: I67a1e987020871b8b9f6707d66b48d9c936e85d1

Merge "Troubleshoot exceeding maximum dispatcher" into tizen

Increase the maximum dispatcher value because added new api

Change-Id: I2f097652775cdc0b42bfd0abd1f58bbe654ef900

Troubleshoot exceeding maximum dispatcher

 - Remove unused variable and change the variable name
 - Check the dispatcher range

Change-Id: Id7d431522f11395b94f1c036726ef3bd862fe497

Change to call gst_init after executing watchdog because it may not respond

 - able to restart muse-server with creating core dump when gst_init really deadlocks

Change-Id: I4946fc6e26ac56d21c747fbd16af80606c5d2d1a

Update to compare the buffer length and the received message length

Change-Id: I0786788de1a0374734e316e2f023494bcabedad0

Remove the dependency of tbm-drm-auth because it is not installed

Change-Id: I4d554513f1b651043f3c3b0c8ecf9a777af9a83c

Merge "Update to wait until the child processe called by fork() completes" into tizen

Merge "Update to split the LOG_TAG explicitly" into tizen

Resolve the build error of -Wswitch-unreachable

Change-Id: Ie4b86754b52f5cddc07d7adb3af0d3bb82ff77f7

Update to split the LOG_TAG explicitly

Change-Id: Ic5c0b5fe24fb6004d6c2e9a83897cedb53025425

Update to wait until the child processe called by fork() completes

Change-Id: Ia55631de7747a91d34a308c46cc22e099cec604e

Update to debug the exception error case

 - Debug reinforcement of exception error of muse connection failure
 - Change the file permissions
 - Move the variable declaration

Change-Id: I9e97c5ad869c54ac19088cb2c7fb60911257c666

Update the static variable of muse client

Change-Id: I13e85154ff2b5739603e2ae6e4cbd3c21820e0da

Update the debug macro of glib

Change-Id: I7a269f7499bd004cb7526e366cc7395336315c19

Update the value of server state when kill signal is catched to avoid the issue of heap-use-after-free

Change-Id: Ib7a280b7d5a7001c21dc14610c74ef90ba5c21e8

Simplifying the return value of epoll_wait using connection state

Change-Id: I10ba91996bfb828f185cd089d72db49940e07e50

Unitest code refactoring

-Remove local definition of buffer lengh and use definition in muse_core.h as using same enironment is always recommended to unittest
-Rename some of macros to have more clear meaning
-Modify read_file to not take size parameter for more generic usage
-Replace printf with DLOG
-Modified to use types in glib

Change-Id: Icd90b52263ff3a89a9e8eb0a8e17dec1b1119d7b

Add the type of vd product by requirement

Change-Id: Iac2a7ad3b0f76460f6423d1b2a17767312bceea3

Add parameter to check buffer length

[Version] 0.3.47
[Profile] Common
[Issue Type] Bug fix
[Dependency module] capi-media-camera, capi-media-player, capi-media-recorder
[Test] [M(T) - Boot=(OK), sdb=(OK), Home=(OK), Touch=(OK), Version=tizen-unified_20180818.1]

Change-Id: I362dad469e259f963076f01928fb25656c5fdf1b
Signed-off-by: Jeongmo Yang <jm80.yang@samsung.com>

Add to raise in signal hanler of client

Change-Id: I237f3b95bd7534a7aaffaba1f4e888d779836287

Add to verify the size of receive buffer

Change-Id: I2fea4c0bb647385e2f5ed0e6783042b424cc07dd

Fix Wformat build error

Change-Id: I1892bba3898d8c29bff7941db875bcd7172fb1be

Update to remedy the epoll_wait fail due to EINTR

Change-Id: I13033d078be6a7795e68bd6c3b4468d83410b1b3

Remove the muse-server.socket because of no on-demand launch

Change-Id: I8cbbff75a3205fa2d51b96bd23bced642a12f8c7

Remove the sigabrt when epoll error case

Change-Id: Ie27b2dd92cbf92768961db689f64b021d79b5dc8

Add the exception log

Change-Id: Iea2e5ff7e5b71366dbe587b3fa345e724a4c4241

Add the on-demand using socket activation with keeping always launching

Change-Id: I5f87d5e44edee733da75a198e64a248e283aacbe

Update to change the registered event with EPOLL_CTL_MOD

Change-Id: I62a43c62c2fe0e93bbceab2afad119e62a056611

Update to log the thread information at the instance create/destroy fail issue

Change-Id: Iceeddf72c8cd8341c3055799859c01c82a0eeef8

Update the condition race of module at dispatcher

Change-Id: I22c92d6eeb16d37ed2822d506d5abf43592daeb4

Add initial gtest codes

 - How to test (json, socket)
 1. build with --define "gtests 1"
 2. Install gmock rpm at target

Change-Id: I70b9dcb1dcc1caeb3d22be9d1749dd87bc486721

Remove the recursion in included header

Change-Id: I66bb914e719cf68bc5d0c11dd75dfb2084a02285

Resolve the GWarning

 - VD's requirement

Change-Id: I4c2d45f147c585bbbf7c7ef1dc77031bb378ed42

Fix the bug of fd stat

Change-Id: I5681556aa21a14dbb2537958a7a01fd9a07f6899

Notify the caution command when the number of instance is warning level

Change-Id: I81fb172d83dbf2b2637f478b10047d1f150fc0fd