Fix typo (subtite -> subtitle)

Change-Id: I159ea077cf79e4a1b3ceca2f51cfd8db35233116

Reduce DB locking time

foreach functions invoke callback while reading DB. It means that the DB is locked.
But sometimes applications do unexpected things in callback and it could cause a long DB lock and
prevent other apps from accessing the DB.
Therefore, saving items to the internal list and unlock the DB quickly.

Change-Id: Ideb57e53a77fa3ace74cec512ca4f3adcf149549

Improve __mc_service_process()

Change-Id: I911f68d4782eab29ec4088d709dcdfc34422f69e

Reinforce message for debugging

Change-Id: I3f72dba1c9392a0e5f1545cb2bd2f371dc2e1c0c

Bug fix of sending wrong message

and change variable from int to unsigned int.
because g_queue_get_length() returns unsigned int.

Change-Id: Iecf364312b725e89ab405ca7e0ad504f09dafab4

remove from the connection list when fail to DB updating

Change-Id: I0ebc9bc99b281e084186f8bea0218f19ee73eb2b
(cherry picked from commit f4a0fe2e8031c113b36b51656e4e43a65c69d334)

build error fix

Change-Id: I1fd9828fa2eb75911e3ab817aa5218bd26544dca

Use proper variable name

Change-Id: I8554da73108c1b37afd2ce4b72c3124479e4ecc0

Improve __mc_service_remove_connection()

Change-Id: I65f5117b11532ed10acc51ac9a911c559145315a

Update log level to get log for product

Change-Id: I9cd72229a715f872af7090b71d6a08a04aeb9a05

Fix document issue

Change-Id: Id86ea1d723f5cabb9cbe9483adda148af2e37b5d

fix memory leak for playlist_name

Change-Id: If662b1a103947c0138b6b515f4d5bf381b239b6a

Fix typo

Change-Id: I00a62ecbe966cfcabafcb4f8b36cb15230318439

Fix memory leak if 'latest_server_name' is empty string.

Change-Id: I9f3af6da75af0d5c77efe2c3c7d7cd0e41aada74

Fix 'sql_str' memory leak if 'handle' is null or 'sql_str' is empty string

Change-Id: I00febeb6285b7b37c6ae3dbb991f75dcb05d7dd3
(cherry picked from commit 5c335a35d6c35c3c3170e6c1584628ceca3af150)

Merge "Fix incorrect return value" into tizen_5.5

Added db migration to upgrade from tizen 4.0

  - minor change. fix typecase int to gboolean

Change-Id: Iad4c2edb632dd76b1564b0704524c6892ad95e9c

Fix incorrect return value

Change-Id: I42af141f7f28fe201a544bb5545168310f46b8c3

Update __mc_db_get_metadata()

Change-Id: I70f1d6370409bac156af0cf28e30d13f6ea91bd9

Fix to return ERROR_NONE and NULL for value when metadata doesn't exist

description for mc_client_get_server_metadata()
"If there is no metadata, return value of the @a metadata is NULL."

Change-Id: I81363d998becd47e7a9c3affc06108f81445405b

Merge branch 'tizen' into tizen_5.5

 - Fix coverity and remove unnecessary codes

Change-Id: Iaa7075c7b972f9b444a506de94f29b45445a4504

Remove unnecessary variables at dbus listener

Change-Id: I0116e1d67200f0e6ece96ca5744534120cfbea47

Unify duplicated code regarding error handling

Change-Id: I85036d0b31671f60345a361df9b3af8efa153711

Fix resource leak for 'query_list' and retrieve 'sql_str_free' function

Change-Id: I247fdeaa76fcbc1aa83c2b85be9da3402e31570c

Seperate parsing and updating database to reduce CC(cyclomatic complexity)

Change-Id: I5087a4bd1970e9f013b2609dd27c1fd544f4ac93

Seperate 'params' variable from parsing and updating database

Change-Id: I99023a723c8c56aaa9f9d9295d5fce2fca7d065b

Remove duplicated conditions

Change-Id: Ia9b557deef1bbc418d16a97e69e63977dbbf243b

Replace 'g_main_is_running' to 'g_main_loop_is_running' for glib update

Change-Id: Icfd3a29e7110cc46bfb8dfbdf496fe4e09482cc3

Fix coverity issue

  - Use after free (USE_AFTER_FREE)
  Returns after free '_dst'

  - Use macro(g_list_next) and add to check return value in loop

Change-Id: Ie08c5fd02a4ad3c80d7d2c054c2ac34cccb3777a
(cherry picked from commit 4a5a45cbd713614071cdf049704c2e5c40b701d6)

Fix coverity issue

  - Use after free (USE_AFTER_FREE)
  Returns after free '_dst'

  - Use macro(g_list_next) and add to check return value in loop

Change-Id: Ie08c5fd02a4ad3c80d7d2c054c2ac34cccb3777a

Change select query of __mc_db_get_record due to wrong result

Change-Id: I82faa60ab52ec8757a92bd7989a1bbc7a80f9578
(cherry picked from commit a138c4535120fa5625dcced6a909ab34dc2066af)

Change select query of __mc_db_get_record due to wrong result

Change-Id: I82faa60ab52ec8757a92bd7989a1bbc7a80f9578

Merge branch 'tizen' into tizen_5.5

Change-Id: I0f26701cea67297bcf78b4560a4f0aef3b0a93b7

Fix build error

Change-Id: I5935f6d070d757baae0ca4b912125c35ebe9c18b

Merge branch 'tizen' into tizen_5.5

Change-Id: I93777ee144e3d263e261a1464d8fd48a7db3f5c8

Version up

Change-Id: I6e9740c833bb3ed436e0a035da3f01ded5e2649b

Apply Tizen naming convetion for protected(global) functions in internal header

Change-Id: I37dcb730361428ff787b3b36bd2fbdc97b35a2b0

Merge "unify duplicated code regarding g_dbus_connection_emit_signal" into tizen

unify duplicated code regarding g_dbus_connection_emit_signal

Change-Id: I36635dc166f5de31b5b282d45bad31e1f1ce3b1e

updating duplicated code regarding getting value from db

Change-Id: Iafd589a36cbfed804c4bb2eb114223097cb76fd5

Remove duplicated codes

Change-Id: I1b1feb67b594544f09a725f4d3248d2c651b4b8b

unify duplicated code regarding command_available

Change-Id: I60137f3cd1efc81f8614a8c2d5913f97fc285cf9

Remove useless parameter checking code

Change-Id: I3901fb99a19be0489743d52eeedd33cbc67b4362

Apply Tizen naming convetion for protected(global) functions in internal header

Change-Id: I1b2d3c26a58f2a82c924f5569938b133cf3eaf52

Remove duplicated code and redundant parameter

Change-Id: I8abe4d8d6eb0f59b6887393126d640bb034bdf24

Fix wrong return value

Change-Id: I6a4935cb30decc1638162cb2ce86cf0926332455

Merge branch 'tizen' into tizen_5.5

Change-Id: I2d96a7a69b77fe187a60b1a2bf1f33f628ed6486

Apply Tizen naming convention for internal functions.

Change-Id: I6dc03a36252d181e96f3774368732f79dda9aa27

Fix to use valid GList in app_dead handler

Change-Id: I752ce18675b1f671e21c323b57a5600e6ea40505

Fix crash while playlist is updated

Change-Id: I16a71c09a3e867a95cdf5b3498ed9487ee4c4c59

Merge "Add coverage exception macros" into tizen_5.5

Add coverage exception macros

Change-Id: Idc1fd2c8b4352451b78400b93ee35c1829bcaf23
(cherry picked from commit 527307c77a36133da7112e46106ab140a24292ae)

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

  error: cast between incompatible function types
  error: '%s' directive argument is null [-Werror=format-overflow=]

Change-Id: I3f8186e0cd44c8e166728e119793ac1eacc95b81
(cherry picked from commit 92f970f92d038a6dc495cca0e67989740095e2e5)

Code refactoring for using GList

 - The loop to find listener over the GList are replaced to 'g_list_find_custom()'.

 - The 'g_list_length()' is not fast because GList does not keep track of the number of items.
   To reduce the 'g_list_length()' call, using index for a loop over the list are replaced
   to using iterator.

   To visit all elements in the list, use a loop over the list:
   GList *l;
   for (l = list; l != NULL; l = l->next)
   {
      // do something with l->data
   }

   reference: https://developer.gnome.org/glib/stable/glib-Doubly-Linked-Lists.html

Change-Id: Ic916d5ef9157b3c7574631b975351507f860fcb5

Add coverage exception macros

Change-Id: Idc1fd2c8b4352451b78400b93ee35c1829bcaf23

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

  error: cast between incompatible function types
  error: '%s' directive argument is null [-Werror=format-overflow=]

Change-Id: I3f8186e0cd44c8e166728e119793ac1eacc95b81

Relocate ability initialization from library to daemon

 - To reduce socket communication

Change-Id: Ice9a2d6212eeb63dfd3f6e7b7b5cb9a34151ffa9

Merge "Remove unused retval" into tizen

Change code not to send msg when media controller server set default values while launching

Change-Id: Ifce00a85e7abd2651e4f668b0e4a7351dc4db5ed

Remove unused retval

Change-Id: I5880aeaaf0fb8cda36492272eb2aa5acbfdd00b4

Bug fix of test code

Change-Id: I8e1c3a0335140923c7259047477e351e448d1d11

Add recovery logics for removed tables

Change-Id: If904c7dc7869962ec7ba8c1ea4433e07d7090c47

[ACR-1444]Add functions for subtitle, display mode, display rotation, 360 mode

Change-Id: I9fd94669658a0de717445f7571003f46f79f96c6

Improve '__mc_client_unregister_filter_listener' function to fix issue

 - Issue
  After filter_listener were unregistered, listener callback was still invoked.
 - Cause
  When all filters were unregistered, filter_interface should be unregistered, but normal_interface was used.
 - Solution
  When filter is unregistered, it is changed to use 'mc_ipc_unregister_filter_listener'.
  After that, unregistering filter will use filter_interface(interface + server_name).

Change-Id: I356c1c0d96fd3297e6f3f61e3b906980d3e2f2ef

Update code to remove SAM violation (GV, PP)

Change-Id: I11345dc1eb6433c91ade42b28321d7354e236bc0

Delete meaningless configuration

If the socket and service prefix names are the same, then Sockets(in .service) and Service(in .socket) are not needed.
Specifying Sokcets and Service is needed for certain functions, so using it without reason confuses the developer.

Change-Id: I6bf969f4a4abc2e59a6885a852bb1909ad861e77
Signed-off-by: INSUN PYO <insun.pyo@samsung.com>

Change mc_client_get_ability_support() to mc_client_get_server_ability_support()

Change-Id: Ifce4e6ec68fcdc4b7a648afc8a4c3b5f366d5f3b

Remove check EACCESS and Fix log position to avoid errno change. (errno can be changed when using dlog)

The media-controller daemon can return EACCESS to the client when the daemon creates a commication socket temporarily.
For this reason, if client get EACCESSn, client must retry the connection.

Change-Id: If7753f8a2590365f4edba2ba6d4183207d0ddf0e

[ACR-1428] Add functions for ability

Change-Id: Ice6c525465f6e741ceb483a39f9c78585c4d3903

Remove duplicate codes for releasing listener

Change-Id: Ib25f7e12c768a9655d7c44b7abbdf9b3429ee1d4

Add to restart service when ondemand is not enabled.

Change-Id: I24340e840cfff21d68a14792d177b29c64207c00

Modify the condition expression

Change-Id: Ie18f8fa16d6ed2ea7399d49c3af97c89979ee243
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Modify subscribe for MC_SUBSCRIPTION_TYPE_PLAYBACK_ABILITY

 - After mc_client_subscribe is called for MC_SUBSCRIPTION_TYPE_PLABACK_ABILITY,
   mc_playback_ability_updated_cb is not invoked
  When mc_client_subscribe is called, wrong user_data was pased.

Change-Id: I1184b5ef76b996b631219d6ccbe9c4ae0f6ba960

Change return type of mc_db_disconnect()

Change-Id: I183a54a6d2154afdaa13e23def10e6c31c87289b
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Add for upgrade from 5.0 to 5.5

Change-Id: I72ad8f0aeafaeeda01f08114fc5a05c90be4345d
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Add logic to apply duplication of search keywords

Change-Id: I35ca0e04e6deebab44ca901c0fcb592989b3bdce

Add static at _destroy_search_list

Change-Id: If1162098d948be58c04d283de4e97e41c59f21f7

Restore removed APIs for compatibility

Change-Id: Ic16ac2081c1b1c20a438fef064835a81a9b7500f

Change the server check for activate and exist

Change-Id: I93199e12482b38dc39e978eac3ff5da8d5f6babb

Merge "[ACR-1411] Add new error for ability not supported" into tizen

search keyword can be NULL if category is not determined

Change-Id: I40a70bb32c17401c6dbedf136fb537527308901a

[ACR-1411] Add new error for ability not supported

Change-Id: I7cd47f659b30873664315fabce17474ef49075ae

Add to free 'msg', 'creds', and 'source' due to memory leak issue

Change-Id: Iea97415f5e4c8abfb963cb5906882dc0dbc6ccaf

Merge "Use __mc_db_get_record() on mc_db_get_latest_server_info" into tizen

Merge "Remove unnecessary setting" into tizen

Use __mc_db_get_record() on mc_db_get_latest_server_info

Change-Id: Ie4a0c2eea5df448f5d554b29e2260606b51fe3e5
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Merge "Add restriction for setting content type" into tizen

Remove check for gcov

Change-Id: I08d37b05d3c89ffd262aeff917db1b0c9a98e344
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Remove unnecessary setting

Change-Id: I583842cf32200d4956abf89b149f4bf379e91b59

Update description

Change-Id: Icdfa353005dfa51ecb312f716215cebcac07545a
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Fix build error

Change-Id: I99ae6d6a8baf61f5dbf687d6f361ae8a85e27344
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Add for line coverage

Change-Id: Iaeb8873f285a16ce1feade31350e40b74deddaae
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Add restriction for setting content type

Change-Id: I214d4930fdf3ec7815906f857ed5bae62a8f5d66

Bug fix of adding playlist item into media db

Change-Id: Ibeeb4937e5aec2f44477443160e0d426bbfeb8ec

Merge "Modify API descriptions regarding the ACR-1355" into tizen

Add error type when no record

Change-Id: I645be2b7262098da16c615f24f383dbc25d63edc
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Delete function for count-only

Change-Id: I01420453bdf44e1f3707ed0fc9095f16d4edeeb1
Signed-off-by: Minje Ahn <minje.ahn@samsung.com>

Modify API descriptions regarding the ACR-1355

Change-Id: I65e03ec0e0c7c18277708cf46d8b2e2e65fec4c0

Merge "Add function for single result" into tizen

Merge "Modify shift operation to use 64-bit operand" into tizen