Remove unused code

Change-Id: I7245d218329b50d757e651c069021b8210a477ee

Fix memory leak for 'path'

Change-Id: I086221beb8558d7bce5b0bc348e96e1a6b46f0c9

Change version of so files for compatibility

- The prebuilt binary which has dependency of this package will be failed
  while loading this shared library by different so name if version of so file is 1.0.0.
  (new:libcapi-media-controller.so.1 vs old:libcapi-media-controller.so.0)

Change-Id: I801c2e5459e03b6b1872e97614d422fb6032edd2

Release version 1.0.0

Change-Id: I4c7e1142e0d78551758323e33e83eb6224c7bf35

Code cleanup for socket functions

 - Remove uncalled timeout_sec
 - Replace return value to socket fd

Change-Id: Ibedbae3cc6d31a8e460964969ae8ac2ade5bb53a

Remove unnecessary socket activation code

Change-Id: I6343c90e712ea4356c9d9c57ac9a022fe50e9641

Modify to limit the size of message for socket ipc

[Problem]
  DoS is available by calling g_malloc0 with too big msg_size value.

[Solution]
  The size of messages with socket has been limited to 10kbytes and
  large messages will be sent as file.

Change-Id: Ie2cf9f4f3a3ec111fcf220724427f29b5157729a

check parameter before using strlen()

Change-Id: Ideff7dbf3212c09653a3262f06b52a147eeece70

Remove unused codes

Change-Id: I3ec6840dafb808932244c58ca33c1b1603e50f67

Bug fix of getting signal name

Change-Id: I6f643fafed6f4469998d91b8d841ed4de73b2de9

[ACR-1825]Remove deprecated APIs

//media_controller_client.h

typedef void (*mc_shuffle_ability_updated_cb)(const char *server_name, mc_ability_support_e support, void *user_data)
typedef void (*mc_repeat_ability_updated_cb)(const char *server_name, mc_ability_support_e support, void *user_data)

int mc_client_set_shuffle_ability_updated_cb(mc_client_h client, mc_shuffle_ability_updated_cb callback, void *user_data)
int mc_client_unset_shuffle_ability_updated_cb(mc_client_h client)
int mc_client_set_repeat_ability_updated_cb(mc_client_h client, mc_repeat_ability_updated_cb callback, void *user_data)
int mc_client_unset_repeat_ability_updated_cb(mc_client_h client)
int mc_client_get_playlist_item_index(mc_playback_h playback, char **index)
int mc_client_get_server_shuffle_ability_support(mc_client_h client, const char *server_name, mc_ability_support_e *support)
int mc_client_get_server_repeat_ability_support(mc_client_h client, const char *server_name, mc_ability_support_e *support)
int mc_client_foreach_server_playlist(mc_client_h client, const char *server_name, mc_playlist_cb callback, void *user_data)

//media_controller_server.h
int mc_server_set_playlist_item_index(mc_server_h server, const char *index)
int mc_server_foreach_playlist(mc_server_h server, mc_playlist_cb callback, void *user_data)
int mc_server_set_shuffle_ability(mc_server_h server, mc_ability_support_e support)
int mc_server_set_repeat_ability(mc_server_h server, mc_ability_support_e support)

//media_controller_type.h
MC_SUBSCRIPTION_TYPE_SHUFFLE_ABILITY
MC_SUBSCRIPTION_TYPE_REPEAT_ABILITY

Change-Id: Ib6a513ac1becb5b172b9022a10e8ee473aca3f60

Improve handle creation performance

- reduce sleep time 200ms -> 10ms

Change-Id: Ie82869950491143d662c2f2e915941206b86be33

Fix build error if build conf is not defined

Change-Id: Ifd2d0d3c096cb355965845917881c6cab8b5f090

Remove useless '#' for functions in document

Change-Id: Ic543e65d1f1cb248ba326e7d47a14ead7a6fbefc

Enable on-demand with socket activation for sec_buildconf_optimized_memory

Change-Id: I0f9be5e23da8a14609afbb579c0665474727273f

Add null check for 'sql_str'

Change-Id: I24f8434bc24cfca802ada8c2c998a75532cca4c2

Remove profile tag

Change-Id: I02b8ec8af626e41b4540bf513d21550886c5c7ae
Signed-off-by: minje.ahn <minje.ahn@samsung.com>

Bug fix. Command or event was not delivered sometimes

[Cause]
All the same items were removed from command list for single sending command.

[Solution]
It is changed to remove one item from command list.

Change-Id: Ic6fea259cdb225aad8a0d267ea419ba026ed53f0

Fixed the build error using gcc 13

Change-Id: Ie340fcdbeea839853e882c374ca0184ab6b2caca

Remove unnecessary 'oldname'

Change-Id: I5dd7ca4116a40165769f1027fd19226b0e779153

Remove unnecessary database check

The mediacontroller resets all tables except the latest server.
If not exists, it will be created.
Therefore, there is no problem when the size is 0.
It is unnecessary behavior.

Change-Id: Icd91f352715cbed9c8499fce4cb5373762162845

Apply mutex when access connected application list.

aul_listen_app_dead_signal() is called in the main thread, and try to acess connected application list.
it can cause a problem.

Change-Id: Ia637c5f4440f25214866e922974e818fc54c9e25

Remove useless structures and use queue directly in socket

Change-Id: I4c2cede650d9c822446d8bba1cacaaac64b67e74

Separate GSource from mc_manage_queue_t

Change-Id: Ib6654861abb1d47dd36ce2556d7bfce125e69fbc

Separate connected app list from mc_service_t

Connected App list does not have to work with request list,
and since it is together, it delivers unnecessary data.

Change-Id: I7d8b1155d6dc400490589fadb00f0ca3819a040d

Modify code to initialize data when it is needed

Change-Id: Ifa177aec2203c11f85b2d3ed66c1b2860a40452c

Fix typecast issue of 'socklen_t'

Change-Id: I6a2d38b71793785e4d755d660154ad8bf59515c6

Fix media-controller-add.post file permission error

 - Permission was not changed to 755 and can't be executed.

Change-Id: I1889a1738d44b71bca061aec5b79b34acc72a37b

Add to create the empty db file

 - On auto coverage tests, db is created as system smack label.
   So we add to create the empty db file as user smack label.

Change-Id: I1acb74576a9d4998d486733440041ca5b03ba0c3

spec: Change the installation path of gcov object

Change-Id: I5f1fadd2ebc4dd088467567e1f87b3831d925beb

Remove TIZEN_FEATURE_COMPATIBILITY code

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

fix 'GSource *timer_src' memory leak and code refactoring for timer

Change-Id: I222811faca43a3a2b29327bb46b324eb49106887

Change gcov object install path

Change-Id: Ib48788cdc9ecadf8c4bafe3a787331df86724960

bug fix. unsafe sign extension of expression 'nsec_t' with 'unsigned long'

 - fix Svace issue

Change-Id: I976dc88152471c5cedddc7138780d5a560528c6e

Bug fix. read memory after free

 - fix coverity issue
  'iter' pointer variable is used after free.
  After the 'iter' was freed using 'g_list_free_full', it is used in 'iter = g_list_next(iter)'.

Change-Id: I70115a6197b7d02ad9afb64376c920a3ea7fcb03

Revert "Bug fix. read memory after free"

This reverts commit d051edd9b2ce8b58e82f90a5defcc3e1800966cd.

Bug fix. read memory after free

 - fix coverity issue
  'iter' pointer variable is used after free.
  After the 'iter' was freed using 'g_list_free_full',
  it is used in 'iter = g_list_next()'.

Change-Id: I156290288f182326d0d44ef120f9d1347adc4150

Fix typo (subtite -> subtitle)

Change-Id: I159ea077cf79e4a1b3ceca2f51cfd8db35233116

Fixed the length of bundle_key to 3-digit

[Problem]
 Playlist is sorted to 1 - 10 - 11 - ... - 2 - 3(character sorting),
 but wanted to be sorted like 1 - 2 - 3 - ... - 10 - 11.
[Cause]
 The bundle_key is character, the length of bundle_key doesn't affects sorting.
[Solution]
 Fixed the length of bundle_key to 3-digit(3-character).

Change-Id: I16b3b59f213f368c9f8203b15acd44b6d370b2c1

Release version 0.2.25

Change-Id: I417f15d060a668876ae166a45e3549d16e4a5d10

Modify the query of getting playlist's item to order by ROWID

[Problem]
 Playlist is sorted by item index. Playlist should not be sorted.
[Cause]
 The database sort the data by item index.
[Solution]
 Modify the query to order by ROWID.

Change-Id: Ie9e21861bf425ee99aae6a16f6d383dce5ea4b79

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