Remove unnecessary parameter for stt_start_file function

The audio format types are not changable in current STT engine.
Therefore sample rate, channel and audio type are not necessary to transmit the pcm data to cloud.
The application should send pcm data, which consits of 1 channel, 16000 sample rate and signed 16 bit pcm data.

Change-Id: I4f9ad034419bf088d6de9d2a1b4a54adfc6e718a

Send pcm data as real time from file, which app wants to recognize the speech

Change-Id: I941d16223458020a546575156a6f5f8917f1a626

Update version (1.70.9)

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

Make atomic variable to avoid race condition

- Issue:
Sometimes, the engine library can access invalid memory location when
the engine app tries to access callback.

- Solution:
Some engine API can access callback function in sttd_engine_agent.
However, these callbacks are not protected by multi threading. Thus,
thee engine app can access invalid pointer if engine library terminates
the engine internally and the app tries too access callback in sub
thread.
Therefore, this patch changes the global variable to atomic variable for
protecting those callback pointers and flags. And also this patch adds
null check for those pointers. Through this patch, engine library can
access those global variables in a safe way even if it is on multi thread
environment.

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

Fix null pointer access issue when check audio_id

- Issue:
The first time to start recording, the engine can get null audio_id from
stt engine module, if the engine app did not set audio id.

- Solution:
Because of the order of calling setter and getter of audio id, getter
can provide null pointer as audio id, but the implementation does not
check it. Thus, this patch adds null pointer checker and fix some logic
to prevent crash by accessing null pointer.

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

Add to set/get audio type on client side

Cause: FFV callback was called, although audio type of sending audio was from Blutetooth.

Solution: Provides a way to set audio type on client side.

Change-Id: If9d2177cfe7aa61bba9fa0480ea750ea9c0d7ceb

Fix to transfer wrong argument

Change-Id: Ic831091feb2d9c092837d3c07de9e29c926f6491
Signed-off-by: sooyeon <sooyeon.kim@samsung.com>

Update version (1.70.8)

Change-Id: I37ac61df076c2f946da5545b7f48056219d1d206
Signed-off-by: sooyeon <sooyeon.kim@samsung.com>

Enable FFV on TV and Use 2nd channel data in ffv audio callback

Change-Id: Idd2514eda85a1896ebd8870a7e5b7e4ab1c77acb
Signed-off-by: sooyeon <sooyeon.kim@samsung.com>

Update version (1.70.7)

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

Extend timeout value for recognition processing

- Issue:
In emulator, sometimes, the client can not get final result.

- Solution:
This patch extends timeout value for recognition processing. Through
this patch probability of getting final result is increased even if the
processing requires more time and resources.

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

Merge "Fix defect detected by static analysis tool" into tizen

Fix defect detected by static analysis tool

Change-Id: I247fc0379958d61a607b077b7f743e322435233d

Update version (1.70.6)

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

Reorder precondition checker

- Issue:
Some unit tests are failed.

- Solution:
The reason why the tests are failed is because the order for checking
precondition is changed from previous code. All APIs first check
parameters and then check state. However, some function check state
first and the parameters after next patch.
https://review.tizen.org/gerrit/c/platform/core/uifw/stt/+/277656

This patch revert the order of precondition check, so every APIs check
precondition with same way.

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

Merge "Fix line coverage scope and test option" into tizen

Add metadata key

Change-Id: I2b3be6281883bc69739159cdef3dc6461331dd49

Fix line coverage scope and test option

This patch reduces the scope for checking line coverage. The line
coverage is for checking client API, so other files in this repo is not
target of the line coverage test.

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

Update version to 1.70.5

Change-Id: I219d8e98f4a4fe9734eddc0163aabc6111f4372b

Extract codes to check retrying dbus request

Change-Id: I98b4b6348f974c82f87dc61f601143730f4b32e5

Add codes to check precondition with state

Change-Id: I474382205ba3f479d7696fed8e06c8f61bfc5a90

Extract to check precondition on engine agent

Change-Id: I21d7729cf7297428ee07be0d79d74f23d2f0a777

Merge "Extract to get base node from config" into tizen

Extract to get base node from config

Change-Id: Icd8c0c2270f7aa3ce1dbdd05606c50078177a31f

Extract dbus_connection_send

Change-Id: I32f75c3f8e98da1e720f3d805985d3f4dcc97c26

Fix value for call by reference and simplify codes for logs

Change-Id: Ie1ff065388e44e7cfa964e545477f28f7d7e0224

Add function to check precondition

Change-Id: If458a13a8f6b937fa3c7c3caab28213e6eadfa71

Merge "Extract and simplify codes to check precondition" into tizen

Merge "Extract precondition for start" into tizen

Extract and simplify codes to check precondition

Change-Id: Id468feec452fc7a8883585258cb8d709a6706ec0

Extract precondition for start

Change-Id: I39dd1765782564a956d7edf92d799aeba2b17789

Merge "Remove unreachable codes" into tizen

Remove unreachable codes

Change-Id: I355058d68d6a2f850ab346dd4e934cad3d85ff9f

Extract dbus message return for reducing duplicated codes

Change-Id: I31a7bdbf6433ef768e4fb65c2a82eee5c14cdfd8

Remove duplicated codes

Change-Id: I8a9b6e53fa371936b50bcfe749ade2dc85c12508

Remove duplicates regarding dbus_connection_send_with_reply_and_block

Change-Id: Iaec79cc7a2801fd10e45349bff77ea96552763de

Remove duplicated codes for set/unset sound for start and stop

Change-Id: I927c0cd1ec1e022a9564a259e82a58d1d2936a11

Remove duplicated code for SAM score

Change-Id: I2ae51acc83aefb32a0eb579ab560c74080c3be41

Update version (1.70.4)

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

Fix unittest for line coverage automation

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

Fix gcov package contents for automation

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

Update version to 1.70.3

Change-Id: Ic2777ff15832c7e0eec2fe643c3856c5a748d6d6

Add metadata profile xml file

Change-Id: I56b09585dc5396e5b46d505f18bf3d43cb2017ee

Update version (1.70.2)

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

Prevent state change even if no recognition result callback is registered

Current code changes the state to ready when the result comes from the server without registered
recognition result callback. However, this behavior can make difference between client state and
real server state.

To handle this potential issue, this patch prevents the state change in this case. By this change,
client state is always matched with server state.

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

Fix the condition for checking dbus data type of uid

Type of uid is unsigned integer, but current code checks whether the uid type is signed integer.
By this behavior, the result from server is not received.

This patch changes the condition for checking dbus data type of uid from int32 to uint32.
By this change, the result is properly delivered to client application by the callback.

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

Update version to 1.70.1

Change-Id: Ia85a00cb18d0b45854a85126bb0d5ab9ed141f19

Merge "Coverage support for robot profile" into tizen

Coverage support for robot profile

Change-Id: I4e18863cac25c9db04442f2801cd09874de30a74

Merge branch 'tizen_6.5' into tizen

Change-Id: Id7b2ac0c4e921eaf1b65efcf67870a693857d450

Update version (1.70.0)

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

Update version (1.65.2)

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

Get UID from list using stt_config_client_s structure

Current code converts the type of element as integer. However, the element of the list is instance
of stt_config_client_s structure. Because the first member of the structure is uid, so current code
has no problem until now. But this can occur problem if the type of uid or the order of member is
changed.

To solve this problem, this patch changes the type from integer to the structure. This can provide
safe access to member.

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

Fix the data type for uid

On 32bit architecture, signed integer is enough to store uid value. However, on 64bit architecture,
signed integer is not enough because maximum value of pid is 4194304 and value of pid * 1000 is
bigger than the limit of signed integer. And it cause overflow problem.

To solve this issue, this patch changes the data type of variable that stores uid.
Unsigned integer can provides enough space to store uid value on 64bit os.

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

Merge branch 'tizen' into tizen_6.5

Change-Id: Ia2573c482c802ef483d6299961d4b98aecfa1ac8

Update version (1.65.1)

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

Fix memory leak when remove item from g_list

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

Update version (1.65.0)

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

Remove access to avoid toctou problem

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

Remove memory leakage to avoid dangling

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

Add dbus_message_unref() to avoid memory leak

Change-Id: Ie9398d91244e804a40569bf5de7f61cb62d0cd79
Signed-off-by: sooyeon.kim <sooyeon.kim@samsung.com>

Change log level for error msg

Change-Id: Ia653a6cc4db506fe4992b462c7bc19de0778d7de

Add bluetooth command to record voice using remote controller

- Add a logic to check whether the current client is TC app or not.

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

spec: remove unused pkgconfig(libsystemd-daemon) build dependency

Change-Id: I3e82ee1f24d96b00b27a49b8b03939f2b7988b3d

Remove unnecessary HTML link for feature

Change-Id: Id588d2e05eeee16fc73ad02427ecfb2ed635742a
Signed-off-by: sooyeon.kim <sooyeon.kim@samsung.com>

Fix Native STTE API references

Change-Id: I091a5da3ee6c00a79ba847bdef86a936ac476a51
Signed-off-by: sungrae jo <seongrae.jo@samsung.com>

Fix defects detected by static analysis

Change-Id: Idff4c4459165f19351b81d1d275a41c7fc498a06

Fix crash when connection was null

JIRA - https://code.sec.samsung.net/jira/browse/TSIX-6919

callstack was as following.

Callstack Information (PID:11475)
Call Stack Count: 24
 0: gsignal + 0x9c (0xb63d3470) [/usr/lib/libc-2.30.so] + 0x30470
 1: abort + 0x10c (0xb63bd780) [/usr/lib/libc-2.30.so] + 0x1a780
 2: _dbus_abort + 0x68 (0xa6a356e0) [/usr/lib/libdbus-1.so.3.19.11] + 0x396e0
 3: _dbus_warn_check_failed + 0x88 (0xa6a2a128) [/usr/lib/libdbus-1.so.3.19.11] + 0x2e128
 4: _dbus_warn_return_if_fail + 0x30 (0xa6a2a830) [/usr/lib/libdbus-1.so.3.19.11] + 0x2e830
 5: dbus_connection_send_with_reply_and_block + 0x214 (0xa6a0ef8c) [/usr/lib/libdbus-1.so.3.19.11] + 0x12f8c
 6: stt_dbus_request_hello + 0x58 (0xa6ab8c75) [/usr/lib/libstt.so] + 0xdc75

Change-Id: I182bda3b921a63a6c43916fe1acb968fb3ebe505

Update documentation according to the review

Change-Id: I62407e2cd0b8b193def6933759c51c93252b5276

Merge "Add STT unittests" into tizen

Update version (1.60.3)

Change-Id: I9631f1d28b73fa7ccba81cef202b26adff1e349d
Signed-off-by: sooyeon.kim <sooyeon.kim@samsung.com>

Comment out TV_FFV_MODE

Change-Id: I25ce7e9717c59062d187dd0e851549fa4edae1a3
Signed-off-by: sooyeon.kim <sooyeon.kim@samsung.com>

Add STT unittests

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

Merge changes Ia10c852b,I6bc34aad into tizen

* changes:
  Change internal callback to static function
  Remove unused code

Change internal callback to static function

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

Remove unused code

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

Version up to 1.60.2

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

Remove sttd_engine_agent.h from stt_engine.c

stt_engine.c produces the circular dependency problem by including sttd_engine_agent.h.

This patch removes sttd_engine_agent.h from stt_engine.c to remove circular depedency.
To remove sttd_engine_agent.h, this patch also changes and removes some functions in stt_engine.c.

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

Remove external definition from common module

`common` module should not have accessed any other modules like `client` or `server`, but it has
used `log_tag()` function on `client` module by external definition.
This kind of access occurs cyclic depedency.

This patch removes external definition on common modules to remove cyclic depedencies.

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

Update version to 1.60.1

Change-Id: Ica951b0b544028b06e0cb63a58e8055231584a46

Fix stte TC fail while tct-mgr execute

Change-Id: I82aa425cd7b89703cd03d86d4c03ea302251d5b6

Update package version (1.60.0)

- Change sound_manager_create_stream_information in accordance with the stream type (from 6df0429105026bc1496e715f0ce2c46b6a8a703e)
- Change a logic for changing system volume using ducking APIs (from f5e97e754e5e8aac40c03291ce14f3cbfe1a37db)

Change-Id: I9add54e3c822b0e1380cea18d64a0ec41a7f99b7
Signed-off-by: sooyeon.kim <sooyeon.kim@samsung.com>

Change a logic for changing system volume using ducking APIs

Change-Id: I7ddf0a47c4ea483f8b8bb19e5957c0ae7595d24d
Signed-off-by: sooyeon.kim <sooyeon.kim@samsung.com>
(cherry picked from commit f5e97e754e5e8aac40c03291ce14f3cbfe1a37db)

Change sound_manager_create_stream_information in accordance with the stream type

Change-Id: Iad4222ae450396f7874a08b322582574ccf03156
Signed-off-by: sooyeon.kim <sooyeon.kim@samsung.com>
(cherry picked from commit 6df0429105026bc1496e715f0ce2c46b6a8a703e)

Add to stop recording from FFV

Change-Id: I4fa1885689bbc690532f21070df10428c96da631

Ignore to call FFV audio_callback when it is not recording state

Change-Id: I110f53faa8867bf39ca072fdcf6971ff954b1abc

ignore audio callback when no stt client is

Change-Id: I00a5262da2425335ea5d2f2145da4851b31945cc

Fix crash issue

Change-Id: I4141e9424a2501eefda98f4b8cee73ba17c1dbc9

Fixed cynara log

Change-Id: Iacaf881fac4b9dcafb24b2f970f374bc16421510
Signed-off-by: sungrae jo <seongrae.jo@samsung.com>

Fixed cynara code.

Change-Id: Ic07a73231e5a2af8c3d8173e1dcaeb07c7d3dfbd
Signed-off-by: sungrae jo <seongrae.jo@samsung.com>

Fix url link properly

Change-Id: I77d9bbde8afe662882597aef1360b360fca3491f

Change internal function to static

Change-Id: I6667d6bbba6c3b57c2caf80a1e48e0ec5f0a6edd

Add function to change system volume in internal header

Change-Id: Ie0e798049c02419db298a028ed243287ac3f6c0e

Fix line coverage script

Change-Id: I11f58a2a75172bec48d5e027c5ff32445d2c829f
Signed-off-by: sooyeon.kim <sooyeon.kim@samsung.com>

Revert "Add parameter for stte_unset_audio_type_set_cb to fix issue for utc"

This reverts commit c482162dd98f4b76f870f6c861f6ca2e3f80ebd3.

Change-Id: Ib6ff0f45d563b452198e35bb46788c99c28526c8

Add unset callback

Change-Id: I27a2e3a237f161159cf440de6d2d3a43da25900c

Add parameter for stte_unset_audio_type_set_cb to fix issue for utc

Change-Id: I76349ec1505904194049b8ab1b4e43a13434035f
Signed-off-by: Sungwook Park <sungwook79.park@samsung.com>

[ACR-1449] Add new api to set audio type

Change-Id: I447e56b41af16cfa116ce07dbb4f3afb33a61aea

[ACR-1449] Add to set speech status callback

Change-Id: Icb3d5c33a3506a453b58ddcd6dab37d9db96b7ed

Merge "Remove unnecessary dbus policy rules" into tizen