Add new key for the bundle of playlist

[Problem]
 Playlist is sorted by item index. Playlist should not be sorted.
[Cause]
 The bundle is parcelable data, the data is sorted by a key.
 And playlist use the item index as the key of bundle.
 Therefore playlist is sorted by the item index.
[Solution]
 Add new dummy key. So it is no longer sorted by item index.

Change-Id: I0cce605dcecdfff9f76a5a51b04e2fff8386ca22

Release version 0.2.24

Change-Id: I6f8ce3bc3de37beea8d879ba990a40d43f2e5ce2

Add retry logic when the database locked

Change-Id: I3f8f6cb05f6f0b53758836aa6ed522b1fafa54bc

Change systemd-devel package name

Change-Id: Ic198c4446203cca6d386f972139085f3ab877798

Update test codes

Destroy client when quit the program

Change-Id: I60c18d185ca7b5e5b9accca09ff82851f6983a12

Add new msg to check service launch status

Change-Id: I9c7148c34e90e5537913b0507e849eaa30d3e2b4

Update code to reduce complexity

Change-Id: I7f2327939287fced4d0b2da1dba6d17d8fd7ebee

Fix wrong messages

Change-Id: I9cdcf62e7eee1844edf0da0fed5ca34db7a2769f

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

Fix document issue

Change-Id: Id86ea1d723f5cabb9cbe9483adda148af2e37b5d

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

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 memory leak for playlist_name

Change-Id: If662b1a103947c0138b6b515f4d5bf381b239b6a

Fix resource leak

Change-Id: I0bfc4257fa11566103d0b4d8068ee986779f2890

Fix doxygen error

Change-Id: Id7c8cec3f416a9a73193b2f62ac10ebca9620cdf

Modify to prevent reuse of values from released handles

Change-Id: I38d37136f45e50d7c5557dea55d097a1d67ad91e

Merge "Fix typo" into tizen

Fix typo

Change-Id: I00a62ecbe966cfcabafcb4f8b36cb15230318439

Add static keyword and apply tizen coding rule

Change-Id: Idd2c7aea4fe8d0ebe5b03d153a214efef4135c74

Release version 0.2.18

Change-Id: I8d672125d14401c28fbaf70a5f08ac292d03c85e

Fix double free if goto ERROR

Change-Id: I6dec59425e7b3851c989112970fb83a30b90b74a

Fix double free in loop

Change-Id: I30a6a9f06458c1c08c11d60ec2897dbfe0f6f798

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

Change-Id: I00febeb6285b7b37c6ae3dbb991f75dcb05d7dd3

Fix memory leak if 'latest_server_name' is empty string.

Change-Id: I9f3af6da75af0d5c77efe2c3c7d7cd0e41aada74

Fix incorrect return value

Change-Id: I42af141f7f28fe201a544bb5545168310f46b8c3

Fix bug that API return invalid parameter for added enum

Change-Id: I9268809c25ad1de446988705f1b0c15713580aab

Fix wrong debug message

Change-Id: I8191d32383bd99c3df17483b7998ff33680a39e1

Fix integer overflow for 'mc_result_code_e'

  - 'mc_result_code_e' enum is used as INT parameter in previous functions.
    Because 0xFFFFFFFF make integer overflow, result_code can be tainted value.
    So it is changed to INT_MAX value.

Change-Id: Ifd8b27ceed31dd55e3e106f1d7a78587354149e5

Added checking valid display mode

Change-Id: I82e2f5688ae7ff54aade4047a19cb49ce93e569a

Fix typo

Change-Id: I6e938c1866a4a09a7d3654c236e80e6ddb7fc5fe

add user_data_free_func for 'g_dbus_connection_signal_subscribe'

  - user_data 'listener' is potentially used in a different thread after user_data has been released
    in another thread.
    so, user_data should be passed to user_data_free_func of 'g_dbus_connection_signal_subscribe'.

Change-Id: I0f7dfc9f338857b6dc0691fc0cd08d0ffa0917fe

Release version 0.2.16

Change-Id: I26e44e22c7974c217145408ab131af9fd4864f9e

[ACR-1561] Add new result code and playback states

Change-Id: I793466c7ea94a6d719fdf8a4ff38329c6d1ad9ef

Release version 0.2.15

Change-Id: I85fdf4b4c2188b290fce9165ed2ca0ebc62cb423

Changes dbus user_data from the listener list to a listener

  - We can specify listener for the dbus signal.
    So it is not neccesary to send whole listener list.

Change-Id: Ia27233c64a897f88933ed2cbb4d97bef6fc81c1e

Code cleanup for replacing client handle with receiver

Change-Id: I52c5cc26eda57cad43f0ef37106c5eece1e79cfe

Code cleanup for function to check parameter validity

Change-Id: Ib6e5a83009c6763ec44abc048031cbd9144e51a1

Merge "Revert "Revert "Fix to return ERROR_NONE and NULL for value when metadata doesn't exist""" into tizen

Merge "Revert "Revert "Remove out of memory related code by using glib""" into tizen

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

This reverts commit 27d487e9fa8ac3033bcdae34dacdf615a0b8ee57.

Change-Id: I43f6a1e248468c8c30bc8ab4814ccd910b969775

Revert "Revert "Remove out of memory related code by using glib""

This reverts commit a54b7e64cc068eafb74493d69119f0ff4dd297c9.

Change-Id: I3b0a9a58471d5c5ab2f46310a950c6d1d31b6563

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

This reverts commit 547866cf02c13ce2e436513dd6fdb62e31fb0e0a.

Change-Id: I2dc463bcd1442740e4735b9f208a5ac66a0ba0bb

Revert "Remove out of memory related code by using glib"

This reverts commit 39b40dbe44d4fd7c1605a78eed918a06abd6eba5.

Change-Id: Ie1fa86f1387cf0f7e5decf340d02dab2cab6dc16

Remove unnecessary duplicated code

when application is deleted from app_list table, it will trigger to server_info and latest_meta table.
if app is not latest, info for that app will be deleted from server_info and latest_meta table.
in mc_db_remove_application(), app_list table is updated by calling __mc_db_delete_from_list() and do __mc_db_deactivate_server().
but __mc_db_deactivate_server() try to remove application info from the server_info table again.

Change-Id: Id4e82f0db44de7b785f25a5d7c30cfce75c791a2

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

Fix typo

Change-Id: I0ca79d1de21c808610432d9758f7764b1afa09b7

Remove out of memory related code by using glib

Change-Id: I42dbbb3e49d0e80b65cb1cfae98a7b76b7812cc7

Remove null check for nullable function

Change-Id: If724198ed0343a528ce749c88042a8e147ec3918

Remove unused code

Change-Id: Iae56504825e2142a6ed2fc4e563150d78d20fb62

Rename and relocate the functions for dbus & socket IPC

  - dbus IPC functions
    all functions except socket IPC functions

  - socket IPC functions
    __make_service_connection
    _mc_ipc_send_message_to_server
    _mc_ipc_service_connection

Change-Id: I7cf1ccb706fa1151a0da963ca338304ff3935451

Remove redundant static function

Change-Id: I952404ccd78ba7eca9c1aae678f9b25c795b6bd2

Remove unnecessary allocation

  - The 'playback' is allocated by _mc_db_get_playback_info() function,
    so it is redundant allocation before calling the function.

Change-Id: I0a6d90db3b85e0b2d841c7cc368bb6cbc74d3b8c

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

Change select query of __mc_db_get_record due to wrong result

Change-Id: I82faa60ab52ec8757a92bd7989a1bbc7a80f9578

Fix build error

Change-Id: I5935f6d070d757baae0ca4b912125c35ebe9c18b

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

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

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>