Fix defects detected by static analysis tool

Change-Id: Icbcddd2e4e1f713cf5a6b1b3b3552d949f25c3a9

Fix defects detected by static analysis tool

Change-Id: I21367d5329158f33548734cd52cb6101e568f2c7

Add log_recorder for printing last log messages on lock-up

Change-Id: I463d0259014a722c0167f1c87a251e28209d325e

Merge branch 'tizen_8.0' into tizen

Change-Id: I9c3fdce18b604c780e3561c4b6815627e360773d

Append custom wakeup words instead of replacing

Change-Id: I2a6d3686dd15efda880312570b7328bed67b85dd

Fix defects detected by static analysis tool

Change-Id: I1782a8acc3e7e4dbdebaa2fd314aec56233592bf

Rename variables to better reflect their purpose

Change-Id: I27dab2104193650c975e21e5213f2625195bd85a

Merge branch 'tizen_8.0' into tizen

Change-Id: I2addb07d38936d0c2eb89ba87030f6a8af35f25a

Bump version to 0.3.40

Change-Id: I0ed70d3b0ee2ee141361e2e522f952eedeb57a7d

Fix build and runtime failures of utc

Change-Id: I2aae00bcf0ec2bb6535fc4353bc220a75a1b00fd

Set explicit names for streaming threads

Change-Id: I1fd74038cc50155093292d5221c9cafac7b8c5bc

Use std::vector and std::string instead of arrays

Change-Id: I5b1a194c39f3f807a039a837efb1e6fafd93f9c4

Directly link wakeup-manager code with executable file

Change-Id: Icb5ca3f5251e57307fb6c6e3741d76f24caaff46

Merge branch 'tizen' into tizen_8.0

Change-Id: I3eff4111818e62eef31f5d842515407ff5ed5a8b

Print logs when getting/setting assistant config values

Change-Id: I4c7339180624144f8b9c2943e91754435f17edf5

Add vconf changed callback for streaming duration

Change-Id: I8e2aeb57f5cd168fa22e6fb8c492adb102c45145

Bump version to 0.3.39

Change-Id: I5e90320d871ce448c55787075da0edcee1518aac

Make sure the process would not be restarted too frequently

If an auto-restart process gets restarted more than 5 times
in 10 seconds, it is considered to be an abnormal behavior
and will not be restarted automatically anymore.

For this reason, modified the app_restart() function to sleep
if the application tries to restart in less than 3 seconds
from the time it was launched.

Change-Id: I44bbb4d24bde28cf35f9927be3dc7bb8c7145f81

Fix pointer array initialization

- Issue:
Buffer overflow can occur when the array is initialized.

- Solution:
This patch uses the zero-initialization for initializing the arrays.
Since C++11, using empty brace sets all value in a array as 0. The
standard names this as zero-initialization. Thus, this patch changes
the initialization using this initialization method.

Change-Id: Ib4362e3db69f2c70cdc16d00ab44303ffbee234b
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Assign return values to individual variables

- Issue:
One variable is used as return value for many functions and they have
different purpose.

- Solution:
vconf_str is used for return values of many vconf_get_str(). It is very
common usesage, but it also cause misuse because developer can not
directly know the meaning of value from the name of variable. So this
patch makes variables for return value of each vconf_get_str() calling
and names them more meaningful name.

Change-Id: I69eb1314795e569d0dc10444aa299380e6412237
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Use reference type to reduce copy operation

- Issue:
When the code iterates the list, some code repeats the unnecessary copy
operation.

- Solution:
Normally, reference type is used when the list is iterated. However,
some code did not use reference type, so those code would repeat
unnecessary copy operation. So, this patch fixes those types to
reference. Through this patch, unnecessary copy operation will be
removed.

Change-Id: Ib64f990c0f33ba5eea2977a80db4765f539c5b04
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

Merge branch 'tizen_7.0' into tizen

Change-Id: If595ae3f0c4d19541ccc75926d69773042ddf5e1

Send restart notification when a scheduled restart takes place

When a scheduled restart takes place on the agent side,
the client application receives a service reset error,
canceling all the ongoing operations that are not
directly related to the multi-assistant framework.
To avoid this problem, added a restart notification signal
for letting the clients to know the subsequent dbus name owner
change event is caused by a scheduled restart event or not.

Change-Id: Id7c3ff404f43e09dfb2488a7402b9afcbc2051f3

Merge branch 'tizen_7.0' into tizen

Change-Id: Ic1b4a88671e616aac03330e747175a111d373209

Disable vconf related code that are not being currently used

Change-Id: Idbf6118351b74ea30e971cf627ac69236f05c685

Merge branch 'tizen_7.0' into tizen

Change-Id: I660f2fc3bc11d35efa5b3a4934efc80d76e277a1

Bump version to 0.3.38

Change-Id: Ibfc511e8953a4d9df1bc86803806f6f7a0f5a283

Prevent duplicated requests from canceling active streaming session

Change-Id: Ie0444f1b7bfa86bec4a415bc393e29ccee5839d0

Postpone restart operation when streaming audio data

Change-Id: I0189262b57d89885c4463a8443939cbc5f67660b

Merge branch 'tizen' into tizen_7.0

Change-Id: I905bc6257b4a774ad8d783bb871ca61cd68d3f5c

Allow audio processor applications to send start streaming request

Change-Id: I36707c55cfc4e64331d2228c30ed051950c15bdf

Postpone voice key release event emission until streaming completes

Change-Id: I5931b2502c06fc645188a20c71a6ea1776ce7e63

Remove unnecessary pid<->appid conversion logic

Change-Id: If0439c8fae6c8ba65d8249eef9dbe4a9ac7d13d7

Reduce blocking time when streaming audio data

When the CPU is busy handling many tasks, there are times
that the streaming thread gets blocked while calling some
undelying platform API functions. To avoid this problem,
applied several methods for reducing the blocking time
and added log messages for displaying time information.

List of methods used for reducing the blocking time:
1. Cache appid / pid information that is acquired by
   calling AUL API functions, since they sometimes
   do not return immediately and take a while to complete.
2. Instead of passing a pointer to the shared audio data
   into the message transmission module while holding the
   lock to protect the shared audio data, make a copy of
   audio data and release the lock immediately so that
   the copy of audio data can be used without the risk of
   race condition. This is because sometimes the message
   transmission also takes a bit of time to complete, and
   the other threads waiting for the lock cannot proceed
   in such cases.

Change-Id: I6c6959fadddbc557dbb77849fa25ce4949155f4b

Prevent restarting service when conversation session exists

Change-Id: I79771d40e67e2bda2efdd6a50474ef500fd5230c

Skip application running check when acquiring PID or AppID

Since the application running check is already performed
when wakeup event gets processed, there is no need to
check the running status every time PID or AppID is acquired.

Change-Id: If93f64f67cfde458123fca6c16f18713fd5118e8

Register package manager event handler before loading wakeup engines

If the wakeup engine app gets installed while
multi-assistant-service is being launched and also
is trying to load wakeup engines, the newly installed
wakeup engine will not be loaded since at the time of
launching the installation was not completed,
and the installation completion event would not be
received also, since the package manager event handler
might not be registered yet.
So if we register package manager event handler first
and then load wakeup engines, even if the wakeup engine
currently being installed fails to be loaded, the
installation completion event is guaranteed to be received
by the package manager event handler, making sure that
the multi-assistant-service would restart for re-loading.

Change-Id: I1e8f57d6a08c9290a0ae11422d1404188c97c3df

Bump version to 0.3.37

Change-Id: I5660de15f66039d338edee05bafbd9d05fe3ec01

Modified not to restart the daemon repeatedly

Change-Id: I4fd0822ac19d5cd010a39afe79410e13f4162805

Change log level for debugging

Change-Id: I3050e5e4e9febd52d2adf131566cb3e90a9438af

Fix issue detected by static analysis tool

Change-Id: I3d296586e161a2a024b301c1ef21e2f406799109

Bump version to 0.3.36

Change-Id: Id0a6a7523197ff665329ab7e5e988c91d6bb1dd7

Fix issue detected by static analysis tool

Change-Id: I573bfa8478cad77c9f38f4eaa00304ea2d854bfa

Merge changes I0271d097,If55e1690 into tizen

* changes:
  Bump version to 0.3.35
  Modified not to load wakeup engine with wake_word_detection turned off

Bump version to 0.3.35

Change-Id: I0271d097f9b17f03899bc7189a41207a065b9fba

Modified not to load wakeup engine with wake_word_detection turned off

Change-Id: If55e1690071f43540e008bead090a6ecb3dc206b

Fix issue detected by static analysis tool

Change-Id: Iba3c2542af7a18ccf2b765a2cc2847c2b5d50219
Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>

Send voice key support mode after changing default assistant

Change-Id: I718cab051170ae75d4ffc49de8675cbb3be969c5

Bump version to 0.3.34

Change-Id: I41e15c0c78de3b8942a946b0868673a51c35fa5f

Modified to load only required wakeup engines

Change-Id: I5088222fbe6217cf8381de5280f552efe3889599

Support checking recording_by_voice_key state for TTT

Change-Id: Ie91e1ece1e9eeb459b1ec18d3d22703bddbc1240

Bump version to 0.3.33

Change-Id: I4c1061b997357760dea6fc78e3cc77a5a748832a

Add missing wakeup_info assignment

Change-Id: Ib192e1c2ade1e922c19f5e5c6ee85431738be3e9

Bump version to 0.3.32

Change-Id: Ia95d45786550312020aefe738eb5e5df04b4bdf3

Make default wakeup policy to choose the first wakeup event

Change-Id: I9c7ca76a001983198d6ffbb458911e28aceffac3

Add the new API for sending wakeup_word

Change-Id: Idfac593bbc829a25036fa9093bb219e157cf1d2d

Preserve wakeup manager state on proactive default assistant change

Change-Id: I772c2000e4201a7798226faeb70279fd36aca92e

Remove media-audio-io and appcore-agent dependency

Change-Id: If9ce14cfe817d90778e2e8c97bfe6dea9b1ca53a
Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>

Remove app manager and network connection dependency

Change-Id: I056f3a8ba87fb63cf1344c27a26319deaf6ec5ab
Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>

Remove unused dependency

Change-Id: I493439e0bd4617fc0f98c1e5f03970d3bcb9e56d
Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>

Merge branch 'tizen_6.5' into tizen

Change-Id: I7d5d642fe7609f656a7e0214ad3136f18a3535bc

Bump version to 0.3.31

Change-Id: Ibcf463383e6c15c3cc9ee4ee2a4d9abdb90b8b33

Add logs for checking if audio fetching is being retried properly

Change-Id: I1cae28820288888d3bba0a75b6e5706244ea7d09

Separate config setter and value change handler

Change-Id: Ia659b1cc4a56eca2a4293493c6ce5c3e21b0e565

Update default assistant appid on wakeup event

Change-Id: I04e3059320e4a9f8e6d4c8902219297d47239a97

Bump version to 0.3.30

Change-Id: I47826ad2f0aa95d062c40d89cc7f0ce3bbd4b526

Fix defects detected by static analysis tool

Change-Id: I34bf30e5c04dc391d185b8f3a8042ebf0530531a

Validate sender information for every request from client

Change-Id: I3d3b51e8b70dec4f8a25eb2827c5f647f6cca273

Bump version to 0.3.29

Change-Id: I697921c3ed2f1169c640d9bd0ae2a7e0ea869fb9

Add metadata profile for providing metadata information

Change-Id: I51a2c31f29c032d68ea7830d7e348b29bd0e6457

Call stop_recording on stop streaming request in UTTERANCE mode

Change-Id: Iea8789ee57a7cb6d64b1e8a47a5735f104e19155

Merge "Ignore voice key press event during follow-up speech session" into tizen_6.5

Merge branch 'tizen_6.5' into tizen

Change-Id: Icf05bbf20fbf9826582438208ea0c35ffe2957cc

Ignore voice key press event during follow-up speech session

Change-Id: Ia48da94500eb9e084fe17431e0b54fe43b9f89f7

Add engine load failure log messages to the ERROR group

Change-Id: I341411c17a6c8511501d240039438df2a41faf31

Merge branch 'tizen_6.5' into tizen

Change-Id: Ifa3a76349c061f47bd67cd733400db6d36a52015

Bump version to 0.3.28

Change-Id: I0769d4d1a80e5527c5c342dc7551ad0d5537e55b

Change default streaming timeout value to 15 seconds

Change-Id: Id5de962d01300918ef7032b1e365053eb5f27c0c

Print memory usage

Change-Id: I95f908f82e9d8a33b24a8710be109d99bfa6c976

Merge branch 'tizen_6.5' into tizen

Change-Id: Idd9643d757bdd1ac297bd1492ecd63cdc5fcb599

Bump version to 0.3.27

Change-Id: I2dad772ca17773518574818eafb4a96d3b7dd54e

Handle default assistant change event in UTTERANCE state

Previously, if default assistant gets changed in
UTTERANCE state, the state doesn't get reset.
Modified to generate recognition result ERROR
event in this case.

Change-Id: Ie9a58cdf8220d5f4d92ac19e79394b7e5145470c

Merge branch 'tizen_6.5' into tizen

Change-Id: Id5816241a659f19ba8862c94f7a672975fd7f1ca

Fix build error in 64bit profile

Change-Id: Iaf71b5f63c361b1942fc9b418720b9fa47aa354d

Merge branch 'tizen' into tizen_6.5

Change-Id: I3aace61b867207bd99cd70cd4cb21670babd2080

Add audio data transmission mileage in thread safe way

Change-Id: I6de9199e4c70ec3f04682a52add02a2164e2eb5c

Execute background volume control command in a separate thread

Change-Id: I1f07ef289514850cc179eb7cd2ff8c16b2918b31

Add audio data mileage and long-running dbus events in logs

Change-Id: Iba38154d8bf3d3bdcf59658679c8b6108a1bc789

Bump version to 0.3.26

Change-Id: I26f1ec4f35754a8ed757609f556c26fdf94a6710

Change localtime to localtime_r for thread safety

Change-Id: I809c8460625f2a10641252b6f0f43ec3853e12e6

Bump version to 0.3.25

Change-Id: I33c503dd82c26130c77ea8613b1f713c29ab5138

Add lock to avoid race condition and add more log messages

Change-Id: I52f7022d9e9ec2df82b09f45e32a7fd8b63aa041

Add more log messages to the ERROR level group

Change-Id: Ic0cffc1ce5f0a422a8947cd50944bbfc40e90526

Send voice key status after recording control

Change-Id: I0df6b18411a218ec5eb271fc6dba6858276db74b

Bump version to 0.3.24

Change-Id: I2bfd26b2f4636651ad802374d13d96ff0879b898

Restore background volume if last requester gets deactivated

Change-Id: I8e70185d093741ff87a7084a915d65453c385f0d

Bump version to 0.3.23

Change-Id: I8517a5b7bb727d671fbdb9c0879977e53ea25f4d

Add retry logic in case open_connection fails

Change-Id: I24abf21cafcadb752f40fc4945114b3bbb2dbbe1

Fix memory leak issue found while running tests

Change-Id: Id4509ed014382aab7938d06ce7920fb9478f8ccf

Bump version to 0.3.22

Change-Id: Ibc1d76df5a8adb85f1bf0f5fbcee1806f5d7ca96