fixup! Create /run/mused for muse server related file

Change-Id: I30d860fdbe394f1818d6dcf03317fc3e02f1dd91

Create /run/mused for muse server related file

 - move file from /tmp to /run/mused
 - rename the hidden file

Change-Id: Id878b8590b2292e6fbdf17cb7661ca2858ab9287

Change pidfile path from /tmp to /run

 - The normal location for pidfiles is /run.

Change-Id: I147f8c09bcf6077305bbbd6fbd4e9e93f6380e9f

Disable MUSE_USE_RM_READY to avoid deadlock issue

 - It is for avoiding the deadlock issue, which is MUSE_USE_RM_READY's _ms_resource_manager_owner_name_changed_cb()
   executing through dbus when dbus error (eg. GDBus.Error:net.stc.iptables.Error.Failed: OPERATION_FAILED) occurs.
   Thread 2 (LWP 2715):
   #0  __lll_lock_wait_private (futex=0xf7567584 <main_arena>) at ./lowlevellock.c:35
   #1  0xf74a0dc0 in __GI___libc_malloc (bytes=bytes@entry=6) at malloc.c:3064
   #2  0xf75d3c64 in g_malloc (n_bytes=6) at ../glib/gmem.c:99
   #3  0xf75fa978 in g_variant_format_string_scan_type (string=string@entry=0xf76e359c "(sss)", limit=limit@entry=0x0, endptr=endptr@entry=0xffdc7e60) at ../glib/gvariant.c:4597
   #4  0xf75faa06 in valid_format_string (format_string=format_string@entry=0xf76e359c "(sss)", single=single@entry=1, value=value@entry=0xf3eb42a0) at ../glib/gvariant.c:4617
   #5  0xf75fb040 in g_variant_get (value=value@entry=0xf3eb42a0, format_string=0xf76e359c "(sss)") at ../glib/gvariant.c:5499
   #6  0xf76ddf4a in _ms_resource_manager_owner_name_changed_cb (con=<optimized out>, sender_name=<optimized out>, object_path=0xf580ec60 "", interface_name=<optimized out>, signal_name=0xe3fef958 "", parameters=0xf3eb42a0, user_data=0x0) at /usr/src/debug/mused-0.3.131/server/src/muse_server_system.c:269
 - Remove unnecessary log

Change-Id: Iaf270a6a9867486f4ff16a5127d4b19e4b61a6af

Remove meaningless log that can be printed frequently

Change-Id: Ib94156237af25edcc5a2e224f459d335f5948f98

Merge branch 'tizen' into tizen_6.0

Change-Id: Ie7ef3053526dc04d34a74dc1c32ef8c007ee954d

Update client log for clear analysis

Change-Id: I61ad8f5bdf0ea003e24b0058c1476a320ca007da

Update log to analyze issues not returned at ms_module_dispatch()

Change-Id: I0feec619b6ce3888ee676ac653b773f5ccced105

Update log to analyze issues not returned at ms_module_dispatch()

Change-Id: I470bb918e164646074fcde7aab9e5972af8c6061

Update to log in case of STDIN_FILENO

Change-Id: I527c7c75065a9807a66b6157f6172f87e2ffc0ba

Change watchdog timeout based on g_dbus_connection_call_sync() timeout of runtime-info

 - Currently, muse server watchdog is set to no response for 1 minute. If there is
   no response while executing the g_dbus_connection_call_sync() operation set to 2 minutes,
   the process is terminated by deadlock. Therefore, change the muse-server watchdog time to about 2 min 20 sec.
 - Add log at ms_connection_lock/unlock

Change-Id: I61a2082c51b1cf3fc24123318a221f4b67c3a4cb

Add log of _ms_resource_manager_owner_name_changed_cb() to check execution complete

Change-Id: I05a004b37002b149aba696fda93af6f27cf89b08

Add log to check if runtime_info_get_process_memory_info() returns

Change-Id: I90d6736c68bba998debe023871f85bb5cd72d1de

Exectue malloc_trim() to release free memory from the top of the heap

Change-Id: I0750d569919faadc9675568a80063116acfa8e6f

Changed the timeout of watchdog to maximum dispatcher timeout

Change-Id: I46d815cd1562448e2640f3d0a7c247663e445419

Update service setting value for the restart failure issue

 - The issue is of 'start request repeated too quickly' and 'Failed with result 'start-limit-hit'
 - Set StartLimitIntervalSec to 0 to configure infinite loop until restart is succeeded

Change-Id: Idb931208a50dc598926831dd244f09b73140ac8e

Fix the duplicate state change while muse server terminating

Change-Id: I729775aa5278bb9a538a22e09b4ac336128a2b93

Update the dlog

 - Set log level for ms_init() to warning

Change-Id: I68d24f8841ebf44c7c500200f02b93c031efaae0

Remove the unnecessary null check related with g_async_queue

Change-Id: I95265493d0a55a63b7d9d459408266f4d0f59bcc

Revise the time buffer array size

Change-Id: I75972a43b18c2322f3b4a834a72037b638bcee51

Change fd dump format of a time structure to monotonic system clock at destroy

 - change log level

Change-Id: I06462498edbdc39342206188677e7a0f3a10859c

Use ms_config_get_host_cnt() to fix DEREF_OF_NULL

Change-Id: I3e977e5f9ff34de2c141887c9c708b2eaba1d70e

Use clock_gettime() with CLOCK_MONOTONIC which is not affected by changing system time

 - update muse_core_get_cur_time() and dlog

Change-Id: I3d194faf73d509d9f438f2cb328a61ba1cad821f

Add gio-2.0, mm-common and libtbm to muse-server.pc

Change-Id: Ic3dd68aa52f6665f28827e0a3aefc885078b7501

fixup! Remove unnecessary dependency for devel package

Change-Id: I1b4bd5f0cd3548075462eea3adb5ec3c07516a4f

fixup! Avoid the module access after destroy

Change-Id: Iad3fbed148a296e6b2e52dd406cfb9e1227662b6

Change to terminate using raise()

 - Remove the unused ms_kill_thread

Change-Id: I6077607177ca5c14bb0480fd05a0630221e10e6f

Check the return value of gettimeofday() and move function execution procedure at connection handler

 - Add connection time log for debugging

Change-Id: Iedfcd2b00816fd7da450869b81f048a463a488a2

Remove unnecessary dependency for devel package

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

Revert "Remove the mm-common dependency"

This reverts commit d67c3e8bb75349b26d1a998791bff9c56ae37730.

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

Remove the mm-common dependency

Change-Id: I37f6a184e77c66117af299d5e92d58e6b40aa5b7

Update the dependency format of cmake as line by line

 - Add missing gstreamer-base-1.0 at CMakeLists.txt
   explicitly to sync with spec's pkgconfig information

Change-Id: I94378e0a653c0d4af84e6947ab9840316441730f

Add missing gmodule-2.0 at spec file to sync with CMakeLists.txt

Change-Id: I88047b4e2a1d5eef2fccfd28ac07b74f04ec4ebd

Add missing gio-2.0 dependency to CMakeLists

 - The build error occurred due to the undefined reference to
   'g_bug_get_sync' without this patch.

Change-Id: I35cb94d30e29cc345b83e95b497a4937a129e1a4

Fix bug to print the idle state properly in every period defined in mused.conf

 - Refresh initial time whenever a client is connected
 - The data type for elapsed time is also changed to unsigned integer to increase its maximum value

Change-Id: I1dfb66666ff130cdda291a710e6b6dff1d39818d

Update the dlog of percent and dlog level related with msg channel

Change-Id: I51a48440378884a3cf82960ba7aed8e0e30ff637

Fix ASAN issue : heap-use-after-free

[ASAN report]
==muse-server==10013==ERROR: AddressSanitizer: heap-use-after-free on address 0xb1baa100 at pc 0xb69cb399 bp 0xbed15dfc sp 0xbed15dec
WRITE of size 4 at 0xb1baa100 thread T0
0 0xb69cb396 in ms_ipc_create_msg_dispatch_worker /usr/src/debug/mused-0.3.110/server/src/muse_server_ipc.c:420
1 xb69d3914 in _ms_connection_handler /usr/src/debug/mused-0.3.110/server/src/muse_server_private.c:312 (discriminator 14)

0xb1baa100 is located 0 bytes inside of 4232-byte region [0xb1baa100,0xb1bab188)
freed by thread T393 (msg) here:
0 0xb6aebee2 in free asan_rtl (discriminator 2)
1 0xb69cac14 in _ms_ipc_module_cleanup /usr/src/debug/mused-0.3.110/server/src/muse_server_ipc.c:110
2 0xb69cac14 in _ms_ipc_dispatch_worker /usr/src/debug/mused-0.3.110/server/src/muse_server_ipc.c:312

Thread T393 (msg) created by T0 here:
0 0xb6aec2ee in calloc asan_rtl (discriminator 2)
1 0xb67fc068 in g_malloc0 /usr/src/debug/glib2-2.62.3/_build/../glib/gmem.c:129
2 0xb69d38a0 in _ms_connection_handler /usr/src/debug/mused-0.3.110/server/src/muse_server_private.c:307 (discriminator 9)

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

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

Do not execute clean-up right after dispatch_no_instance without destroy dispatch

 - Add log of module pointer

Change-Id: I7503b987805cdd6b9fd27add9e19841425b469bd

Change thread name

Change-Id: I686120fbe819f62327fc34611c6c2de11d3e95b0

Code refactorying

Change-Id: I08eb8d84e24cd8ed853f473c78ac2789d2fecdf7

Revise the cpu usage

Change-Id: I52a833c55dc758dbfc23ba797e68cce184c24ee5

Update the diagnostic thread structure

Change-Id: I223d6e2840d84aa88899045aa7e2d5507a5afbad

Fix the issue of buffer overflow

Change-Id: I4ddd17cebc7afa198740dbee9e7f2ba4ce9905fb

Use g_aync_queue instead of epoll for handling of instance connection event

Change-Id: I3f801559dcc202c68b761a062e04263a73f1c11a

Set high priority of g_idle_callback to create ready file

Change-Id: Ibf56dfd200c7c10936d70137f0c32538609e6335

Check the value at idle time for debugging the cpu usage issue

Change-Id: I0f7a411585e127952eb635a5127448452a413e40

Create diagnostic thread after executing g_idle_add normally

Change-Id: Ic1cd606923dc57f042ae06abfe0ae13fcfdc8e4f

Remove the unnecessary g_thread_exit call

 - Remove warning log

Change-Id: I32e4faccc901f877048589aaa48456a4e89b017c

Separate threads to facilitate control of each operation

Change-Id: I3f7aa905fb4693dee3af180a8ab77c686ec06e49

Print the log of the instance information at regular intervals

Change-Id: Ibaeb74932b5e20ad1cc872a75b6fbfa24a345259

Add the coverage measurement for muse

Change-Id: Ia8a2594517622e7685d5fcfbb283466f73e97f30

Change the variable of extern with static

Change-Id: Icbc7f2e3dd946926fd728e4176316837dd1bdaf8

Use g_new0 instead of calloc

Change-Id: I37f4ca0cb09fb52d2f7e960679cd53784736e465

Update the argc of gst_init_check

Change-Id: I40322da038e328cdffa660bef3a9c75a6788eac1

Use g_malloc0/g_free instead of malloc/free

Change-Id: Ibefc3b87ed4f5cfeb4e97cceb5e7c819420acf09

Remove non async-signal-safe function in signal handler

Change-Id: Ia092ae0515d4d28e58d248b041a0944c26a0b9df

Fix the typo

Change-Id: I13fa4503fa094b8094b4434b7c0c7bbe1d4557ed

Change remove with unlink of async-signal-safe function

Change-Id: I9851fccfb38337802ceb59bf96ff32d738a691dc

Revise the watchdog period

Change-Id: I13bf84b3bdb66bfca68ceeba64257d71d411f885

Update the macro function

Change-Id: I19ef9fc6fc3032b051a802c5eabd2d11e9afae89

Merge "Do not check the value of log_enabled which is always false now" into tizen

Do not check the value of log_enabled which is always false now

Change-Id: Ibc7a77bdf9de8858a6a59ddee6970b03daa07588

Merge "Fix the bug of signal handler" into tizen

Fix the bug of signal handler

Change-Id: I296c472586bd30e01faec2cd4679377bcec995fd

Align with a fixed tab space

Change-Id: I8c412596867a764ad3d264f65bf6bebd96404dbd

Do not call some debugging implementation of deadlock to reduce executing time

Change-Id: I287ab842f2388e49c9da39a04546e50a573b0da9

Disable the signal handler of client

Change-Id: Ib17a9424edd893f96f757d80cd627d6199354a29

fixup! Send sigabrt to debug what the server is not ready

Change-Id: I1064839f86a9f11f86c9fb764cb4304c9b5cc3a3

Send sigabrt to debug what the server is not ready

Change-Id: I29b6da8faaa395381094a20ee27faa3357ca7a45

Revert "Call shutdown command explicitly to release client resources in signal handler"

This reverts commit ebde40ec0da37da3239506d9812b0d181a9ef83e.

Change-Id: Ie74328f78385b3100bd8e9f18ac74a8ab8c43ca9

Fix the bug related with muse_server_is_ready()

Change-Id: I8b0474bb7cecb5422328dbf56c9de38cc22faafd

Call shutdown command explicitly to release client resources in signal handler

Change-Id: I8112fc27468dc15a194755e8f2020d9bc879fd88

Add feature for preloading plugins for GStreamer

- Currently, we should describe full path of plugins in mused.conf file,
  but, this patch simplifies it like below.

  As-Is:
  gstparam3=--gst-plugin-load=/usr/lib/gstreamer-1.0/libgstcoreelements.so

  To-Be:
  gstpreload=coreelements

[Version] 0.3.103
[Profile] Common
[Issue Type] Update

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

Return FALSE so that source callback is not maintained when an exception occurs

Change-Id: I5bbd839be999796435db171eac753be974e23801

Change to preload module after gst_init call

Change-Id: I2dbecf832745e0e5d15d7efbf820a08aab7b220f

Update coding rule of muse

 - Set max colume value as 128
 - Set tab space as 4

Change-Id: I569b7f7e69be28c1b7bbdbaeb1af89d274e1f5f6

Add api name information in dispatcher deadlock

Change-Id: I4e8a0e182369bdd4a42890e0b872b17445ae6afa

Making variables to static

Change-Id: I578014b01c3d50502aeca8aceb043d07c5d8c4d8

Fix SVACE (DEREF_OF_NULL.RET.STAT) issue

Change-Id: I1cfb26c4308b296144624dfa5049a5a2e5522053

Fix build error due to iniparser upgrade

Change-Id: I4cb53b2741448854368607f03fc8c32bcd929f8c

Merge "Fix build error due to toolchain upgrade (gcc6->gcc9)" into tizen

Fix build error due to toolchain upgrade (gcc6->gcc9)

 - Werror=format-truncation, -Werror=cast-function-type

Change-Id: Ieaa02eca90e455610bfea45cfb88ec436faefaec

Change the return value to avoid g_source_remove at connection handler when muse server is not ready

Change-Id: Ide6274b8c67bb10d661ba5c609dea3b623c2ffec

Merge "Improvement variable initialization" into tizen

Fix the bug of debugging information from fd directory

Change-Id: I087db4ba66f13c7b7736ec359810d973c40535f8

Print the process information in dispatcher deadlock

Change-Id: Id3cc060b5e74b4cc8dcdb8cd19fcac2630c61750

Improvement variable initialization

Change-Id: I6e27d54859c70120297fe4aab9ba78ce16a566d9

Change the value of timeout in case gst_init does not return

 - Create the server ready file with g_idle_add
 - Update to check the 'remove' return value and add errno for debugging

Change-Id: I1d92c7b28fd502e0d2dafce4754d0acbe251dec9

[VD Req] Do not execute mused in standalone mode

Change-Id: Ic79c5a91fe890b7ec016b1cb760217f4ce289c8a

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