Fix svace issue of PASSED_TO_PROC_AFTER_RELEASE

Change-Id: I24952080e6c800d393d257a06369bef528a671e5

Apply CPU inheritance boosting to improve connection performance

Change-Id: Ic5118202ffeb933f128c55985363425a3c6cc532

Improve VD muse-server.service initialization

 - Remove standalone part
 - Set default dependency as 'no'
 - Execute after sysinit.target
 - Avoid cpu contention using boostshell

Change-Id: I1305541e365c38c8906fa2217bf340154a9533d5

Remove unnecessary path file

 - There is currently no dependency on tvplus_ready at VD

Change-Id: I9e2be4e3e5c38c1f1e0edaba2b70c9435111ed61

Add to detect stdin close

 - There is issue of accept() 0 after idle, which means stdin is already closed as below.
    muse_server_private.c: _ms_check_idle_state(523) > [#3] 2707s [period 900s] [0d 10h 8m 56s]
    ...
    _ms_connection_handler(376) > Critical Error : accept 0 is invalid

   So I add to check periodically the status during idle and after destroy() to debug issue.

Change-Id: I866e638c9a9a98b74fdad58f963c7cb39c14b7ed

Revert "Disable system resource boost"

This reverts commit 638fd25b1db57654921968a8fa6bca0248b856fb.

Change-Id: I6abc61876dfdbfa147095728eba341d358cf1b76

Disable system resource boost

 - Recently, we have applied a system resource boost for service launching performance
   despite the restrictions (client connection issue during cpu boost) by VD requirement
   but the restriction issue came up as expect. Since I disable it at first with keeping
   the relevant code rather than rollback a patch.

Change-Id: I866342f8d526b39b1ff49d8a46deaeb197e14d9e

Add system resource boosting API at VD

Change-Id: I328e47ebbd5c46e0a7cd67b8babf44387d603a72

Exclude storage dependency for VD

 - Remove unnecessary parts
 - Revise the same flag name of the cmake and source file

Change-Id: I1df397e258c932b1b8c3df16e26ec1999e3eb474

Add ISU package

This commit adds Individual Service Upgrade (ISU) feature to muse package.
When installed it will allow to upgrade this service using ISU mechanism,
without the need to perform full OS Upgrade.

This commits add necessary configuration to create upgrade packages of
needed format.  ISU framework itself is maintained and installed separately.

Change-Id: Ib30741bf33cb82d4d8565bae2b65520f3e9b430a
Signed-off-by: Mateusz Moscicki <m.moscicki2@samsung.com>

fixup! Set simple type at tv service

Change-Id: I866524a6e61f224ea72487cec152c9285fec1016

Remove unnecessary comment at README file

Change-Id: I57377dbc627fe7c8a190b150875bb44d85b9676d

Change name for threads

[Version] 0.3.170
[Issue Type] Update

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

Revert "Install the notstrip file to avoid be stripped at vd product"

 - vd requirement which 'not stripped' is not need now.

Change-Id: Iad6d111a1803a88c92da534cc0f621a078534cbc

Exit process in forking error case to avoid systemd timeout

Change-Id: Ib2d183daa28f48e12824f50ac78b989fac4627a2

Fix coverity issue of OVERRUN and UNUSED_VALUE

Change-Id: I62c2a7a4c46bd5fb77fef1b89dd7731ea2013c05

Set simple type at tv service

 - Avoid netlink event delay issue of parent or child process by daemonzing
 - UPdate dlog

Change-Id: I91a9d22eddd893d94d12cac3873d87f73da06e7b

Revert "boost muse-server for PNP mode at tv profile"

This reverts commit 5630fb38335f5b2e32b292f1e7abba6f178a4baa.

Change-Id: I4f78a9e8f6dcd4684d8d16929ae9f44f19f7d3b6

boost muse-server for PNP mode at tv profile

 - update log level

Change-Id: I794beb57e84503287165fc426e24ed7053663f89

Fix svace issue

Change-Id: If274f4d84044220e9543991f49f073291a893da7

Apply with GMutexLocker for code conciseness

Change-Id: I76594247fc8bb59d8726c82cfd68487050ddc478

Check the stat file again to make sure it's cpu busy

 - CPU utilization, as some tools like pcup may not be actively maintained or may not be accurate on modern systems.
 - The top, mpstat, sar, and htop commands are commonly used to monitor CPU usage on Linux systems.
 - So I have updated to check again the /proc/stat periodically after new timer creation considering performance

Change-Id: If76612dec2ed201abe617bbf4c7fd192a979f2f0

fixup! Fix heap-use-after-free issue

 - By thread scheduling, the data thread can be executed before setting the value of 'is_created'.
   where is set at _ms_ipc_dispatch_create().  So I remove to check value of 'is_created' which cause
   it not to run despite normal data thread execution. Instead we can use muse_server_module_is_valid().

Change-Id: I8bc4c0ac65d4ee98e477c0d5e475c2c60cff289e

Print socket fd value at dlog when send() failed

Change-Id: I5138e668331290b6b54ad0e13056decfd91138df

Fix heap-use-after-free issue

 - After module's msg thread exit, the paired data thread could be accessed
 - Add to check if module is freed or not with module's memset variable

Change-Id: I26f18ed896420915f1cdab2b3f03f15d137a1436

Check module's idx value to avoid dlog crash

 - We have issue of dlog crash when the value of module's idx is -1.
   Unfortunately, there is no log file, so if module idx is abnormal,
   we do not take the log at first.

Change-Id: I051871c96c71aacd8d130b5b110ff72b19e2620e

Fix heap-use-after-free

 - Module released by _ms_ipc_module_cleanup() could be used again in ms_ipc_create_msg_dispatch_worker
 - Put into protected critical section for ms_ipc_create_msg_dispatch_worker()

Change-Id: Id68e77b5da8db91f132429bed230be03895c6857

Fix Svace issue of SEC_DO_NOT_ASSIGN_SIZE_OF_INT

Change-Id: I967f1b9c08818c86de2b5c225e5b65fb5a461386

Fix Svace issue of SIGNED_TO_BIGGER_UNSIGNED

Change-Id: If3e347df3e06b554d719c7e9361f688ef0e5a314

fixup! Use PathExists instead of ConditionPathExists

Change-Id: I614d83c5dc61a1a128dc871379dd590bb165035a

Use PathExists instead of ConditionPathExists

 - ConditionPathExists does not guarantee service file execution sequence.

Change-Id: Ie29ee1954aed23eee2bc931ffdefaf3795a20b7b

Fix asan issue of heap-use-after-free

 - Module released by _ms_ipc_module_cleanup() could be used again in ms_ipc_create_data_dispatch_worker
 - Put into protected critical section for ms_ipc_create_data_dispatch_worker()
 - Update dlog for debugging module handle

Change-Id: Idcbb2efa5ab10a176128d0254613cac6cb85c2dc

Put into protected critical section for ms_security_is_privilege_granted() deadlock issue

 - cynara_check() is not thread-safety
 - Change return type without checking return value

Change-Id: I4d1c650e2284ce35a860898a99c4ba473bfc2139

Fix coverity issue

 - FORWARD_NULL

Change-Id: I85c115161d608604c3b5679a7e9c0adc7c42c397

Do not exit at dispatcher worker when destroy dispatcher is failed

 - The dispatcher worker exits when socket is disconnected and then module_cleanup is called.

Change-Id: I4be7b552f44aac331c907e8fc21d121e310ee484

Add muse_core_set_close_on_exec for FD_CLOEXEC flag

Change-Id: I31b83574c2fbd4119334ff7124eef8321f803f05

Check for suspicious memory leaks on every dispatch

Change-Id: I60cb4b5bd0059a0e8375c71d03461ba7dfb51d6b

Fix svace issues

 - Unchecked return value from library
 - Remove unnecessary header

Change-Id: I9a31cb772f6fdd7a64b006488091830b8502daf0

Use ConditionPathExists to run the daemon only after the wayland service

Change-Id: Ic24633f583067fcdfeac2f5e421e3c297331ed69

Set whether to measure the memory through the configure file

 - It can be disabled according to needs, such as performance issues

Change-Id: I937233f4cd3fb3d0bfdad23783c21beba972533d
(cherry picked from commit 6da42e63e253555cbf36322cd86804dbf791f04e)

Fix 64bit build error

Change-Id: I33a3fbf3ebf8bded984f69655fce39cf71005867

Remove void casting

Change-Id: Ia25b747f5cb6d2adffccb54028e2087ffa33da2a

Initialize the buffer manager at server startup

 - Reduce tbm_bufmgr_init() execution time at client’s initial create()

Change-Id: I78dd1cdd9fec0af83cfad97f482ed969eea20a8b

Remove the duplicate variable of server structure

Change-Id: I8d1259d42e5a4056dd7991d0ab6eed936d37c298

Set the mmap and trim threshold of 64-bit

Change-Id: I759acb585286257d3660dd69e9a67eb6d050e361

Add libtzplatform-config at dependents explicitly

 - Build error occurs because libtzplatform-config is removed from system-info at VD

Change-Id: Ib8361fadd71e5c67854f6fe236bd7fe9d7600069

Remove non async-signal-safe function from signal handler

 - As Is : Set IDLE state with g_mutex_lock to avoid new client connection while signal handler executes
 - To Be : Remove g_mutex_lock and dlog which deadlock can occur by non async-signal-safe function

Change-Id: I665ebe5017669530912ce493fa5ed9cb378dc19b

Change path of configuration file

Change-Id: I1ea1a26bf85cfc244ce7b74385293c07b446738b

Fix coverity issue

 - Improper use of negative value (NEGATIVE_RETURNS)
 - Logically dead code (DEADCODE)
 - Out-of-bounds access (OVERRUN)
 - Data race condition (MISSING_LOCK)
 - Update strncmp() part totally

Change-Id: Ie6b23fb1281563522183a90e81048d15cea5207f

Change the return value to execute the dispatch only once when no instance

 - It is to prevent unnecessary recv() after dispatch is normally completed.

Change-Id: I3875f65690a1faa4257ce7d2dd55c798f28adc03

Change systemd-devel package name

Change-Id: Ia602785781630caf42acd0c503e1c03ccdf2f425

Remove the dying message feature of deadlock's reason

 - backtrace_symbols calls internally malloc of async-signal-safe
   which is reason of deadlock when crashed

Change-Id: I47ed587be71cbf291b9c8df0a15cbe8d40767a7e

Fix build error of muse client

Change-Id: Ib7d19124e29a50aea2aac6f40681213588a8a545

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