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>

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>