Bug fix. disable product_tv as default

Change-Id: I07ba95262a2fa7eb96d71f76f356b65a130f788f

Create a file to verify controller daemon has run

create tmp/media_controller_launch and remain additional message when socket connection failed.

Change-Id: I8b324119f9a2f944f7db85cea3f4a885af20fc27

Bug fix. insecure strncmp

 - fix svace issue (LIB.INSECURE_STRNCMP)

The problem that such using checks only prefix of string because null-terminator is not checked.
It may be source of vulnarability when using for compariso passwords.
Correct pattern should use strlen(arg) + 1.

Change-Id: Ic9ac12468a6c81901e1c169097109e47a4c496fa

Add more info to debug message

Change-Id: Iba87610cd54d189598e263960bb4c947a6ba6b1d

Add OS upgrade script for TV

Change-Id: I58c094dadc787c72431087cb9225e19d5911e1e8

Improve API description

Change-Id: Ieae51c84ef95a6f93cfe004c9f96b4b7d540334a

Code clean up regarding the return value

Change-Id: Ic33390455a90e7649a6d00a5b0870ac99452495f

fix doxygen header issues

Change-Id: Icd85f2a213fc6e6891f562c87243b90a6b9874d3

Bug fix. insecure strncmp

 - fix svace issue (LIB.INSECURE_STRNCMP)

The problem that such using checks only prefix of string because null-terminator is not checked.
It may be source of vulnarability when using for compariso passwords.
Correct pattern should use strlen(arg) + 1.

Change-Id: I3fd5a9e785e5f87a53b23909ddc01963dbd8916d

media_controller_util refactoring

1. Checked only an app name not a full interface name.
2. Changed the substitution pattern from 'APPID_SEP' to '__'.
3. Used g_autofree.
4. Used g_uuid_string_random() instead. (Remove uuid dependency)

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

Remove unnecessary executable permissions

Change-Id: I8985201af0205bafb8c56c16fd3071542cec9339

Use g_auto(GStrv)

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

Add sql_autoptr type

sql_autoptr:
__attribute__((cleanup(sqlite3_free))) char *

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

Remove meaningless code

This code is meaningless for the following reasons:
  1. If there is no db file, media_db_connect() creates one.
  2. 'pragma integrity_check' is a command for checking consistency of db.
     Therefore, it is independent of file size.

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

Use sqlite3_str instead of gslist

sqlite3_str makes playlist-related operations simpler.

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

Remove unused variable

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

Use sqlite APIs directly

sqlite APIs already checked input value.
So no need macro.

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

Remove db upgrade function

DB upgrade functions is for tizen 5.0.
No longer in use.

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

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