Fix BAD_CHECK_OF_WAIT_COND in cache agent

Change-Id: Ic7db1689563b71d885b9607b7268db93757f09b7
Signed-off-by: Jaehyun Kim <jeik01.kim@samsung.com>

Fix BAD_CHECK_OF_WAIT_COND

Change-Id: Ic51fa5b8cd1db7f122bb126c1cb4c78e339e9ab7

Fix calling of g_string_free_and_steal()

Change-Id: Ibce062f6601eeb470c65c16ad9dc144779e65f50
Signed-off-by: JinWang An <jinwang.an@samsung.com>

Use g_string_free_and_steal() if glib is greater than 2.76.0

Change-Id: I11d2a944fbde6040700c21ac4bfe8caf41eb5472

Do not use POST method

Change-Id: Ia193d9b79b810c1066582fab8f14aac4a5ae9327

Modify CURLOPT_HTTPPOST to CURLOPT_MIMEPOST

Since CURLOPT_HTTPPOST is deprecated option in libcurl, build error
occured in Tizen-Unified-X environment.
According to the documentation of libcurl, CURLOPT_HTTPPOST is replaced
by CURLOPT_MIMEPOST.

Change-Id: I9717a27967c171dc6609e55fcc6470e4c9c21400

[CID-1745069] Fix data race condition

Change-Id: Ib4024a3d165b6c576dd147329f0485396ca9eecd

Remove some lines related to openssl

Change-Id: I7d65482d306cc90742e435a115adaa9184932db8

Remove notstrip file

notstrip file is no longer used.

Change-Id: I0c5737946ed7033a50a3f090a33b6501f06f417d

Add cache support

Cache Agent and related DBs have been added to support cache
and logic using them has been implemented.

Change-Id: Ieaa8773959d9c3c6b0b0b2c8c90da243452a677d
Signed-off-by: Jaehyun Kim <jeik01.kim@samsung.com>

Check if temp_id is overflowed

Change-Id: I1d1aad6ea6b2c46ba037afd5fa224c391614e3d5

Check if parsed_url is null check before using it

Change-Id: I45b13728b1b73c0cd06f317c1e4e8dd9f8dd575b

Fix for svace issue 518382

Change-Id: I22239fec80cae6799529cc3627091beaefcdcaa1

Check both CURLINFO_RESPONSE_CODE and CURLINFO_HTTP_CONNECTCODE for HTTP status code

Change-Id: I1e1f7186d743d2bc8da4b33834d3f67e5698e3ec

Merge "Accept spaces in URL" into tizen

Merge "Check a exception proxy list before transmitting HTTP request" into tizen

Resolve probable infinite loop issue due to integer overflow.

Change-Id: I7ad2bc1d6ef28510ed6f17fb004425dca4637a56
Signed-off-by: Akash Kumar <akash1.kumar@samsung.com>

Accept spaces in URL

Change-Id: I1a9023e77596c04b736ce56b21eb1e4a6c70a30f

Merge "Reset sockfd after close" into tizen

Check a exception proxy list before transmitting HTTP request

Change-Id: I3e54200a9dcb34001ae8e7d6f9a188fabafe3b11
Signed-off-by: pyun <dh79.pyun@samsung.com>

Do not use CURLAUTH_ANY

Curl known issue.
https://curl.se/docs/knownbugs.html#libcurl_can_fail_to_try_alternat

6.6 libcurl can fail to try alternatives with --proxy-any
https://github.com/curl/curl/issues/876

Change-Id: I31ce09e8bb3c8286804224df23e4fbfdba663aa5

bug fix: handle empty proxy address

1. Revert e09e63a4a974ec9e6f2c094d7562bc89cb3452f8
2. Exception handling for empty proxy address

Change-Id: I012edb83c505cc8c17d6078dfdaa75c291164112

Fix the download fail issue on the normal environment

Change-Id: Ib6ddcbe831dad72e5188f44a598cb3bd5d847bd8
Signed-off-by: Dohyun Pyun <dh79.pyun@samsung.com>

Get status code using CURLINFO_HTTP_CONNECTCODE when using proxy

Change-Id: I93309e8c3ba38e0e7c70b389ac8c6e6caa80b258

Reset sockfd after close

Change-Id: I31301b6646c36143f4c4c3583c119bb35e50bc37
Signed-off-by: cheoleun moon <chleun.moon@samsung.com>

Use the same type to compare values

Change-Id: I98c2c1bb5ee85fe15de100340c19a00530d7f3cd
Signed-off-by: cheoleun moon <chleun.moon@samsung.com>

Fix svace issues

- SEC_DO_NOT_USE_INT_IN_FOR_LOOP
- SEC_DO_NOT_CAST_LONG_TO_DOUBLE

Change-Id: I0b0fb1f1ccdd00f0281cc404c153cf35e58a7d7a

Change the naming rule for duplicate files

Change-Id: I25a3010f7e3b8f76deccbfa44ac08966b9fc8441

Increase max header length to 4096 bytes

Change-Id: I1427ebddbd1f564eef1879ba922b39af96ad5608

Retry read() when getting EAGAIN

When download is requested from new client, client DB including several tables is created.
This often takes long time on RPI4 and makes IPC error(EAGAIN) in the end.

Change-Id: Ib28dcc22e929d11fe61e5f1690f7da64ee5465b8

Do not change a permission of files stored in external storage

Change-Id: I1971284beff321a28d50b2e4769cdcf72a441eab

Change ownership of downloaded file to client's uid

Increase version to 2.3.0 for Tizen 7.0

Change-Id: Ib4c79ee3b20404546a0e459d8964635f143de755

Fix NULL pointer dereference

Change-Id: I9d2cf34fc9e924da5dec01d5229c4b7532b35f48

Retry read() when getting EINTR

Change-Id: I476b3db2a3dfb6519d885c67c052914201eeca4c

Remove stale data from a client DB when client is terminated

[Problem]
a client DB size is increased unlimitedly when a client is terminated by
itself or a crash.

[Cause]
a client DB is cleaned only if a client is live and has no response
during 2 minutes.

Change-Id: Ib164be4cf446bd6c30b0bc89b50d902b0cc5955f

Remove an error code to be overwritten

Change-Id: I4f582c81220d42dcf8fb7945b168a439fdcf59f2
Signed-off-by: Cheoleun Moon <chleun.moon@samsung.com>

Add missed unlock

Change-Id: I9ad8f4f9525e2f9c8f5b43ec7994469a6c68cd4f
Signed-off-by: Cheoleun Moon <chleun.moon@samsung.com>

Avoid to use stat() function

Doing something after checking of the state of a file has a potential
race condition bug. It is more effiecient that just using execption
handling instead of checking to prevent TOCTOU.

Change-Id: I2a2560d9d49ed9d52008557f0d2f89fa04bb130d

snprintf(): use sizeof() for size factor

Change-Id: I9a94e6c82b1bd40f06d66e8e2bec13a860ce5f20

Change systemd-devel package name

Change-Id: I90d548f1f86be4df188fff33d291b6122edbe939

Change the type of some log messages from DEBUG to INFO

TV image doesn't show DEBUG message.

Change-Id: I6febbb33c936bbb7f3d9440faf8e01681b842179

Limits concurrent downloads to 15 on TV

Change-Id: Ib70042cd7c9e6f875e7c32cef76e434ec7d6e77a

Do not free http_info in client request thread

http_info is used in download thread and it is freed if download is
finished.

However, in pause state, client request thread releases http_info if
client requests 'cancel'.
This makes a crash occasionally during download thread is terminating.

So, download thread should have ownership of http_info and this thread
should free it only.

Change-Id: Ia3ec64d1ce64e2aee49e260b901686a4ae6d4d76

Fix heap-use-after-free

In case of below scenario, da_info is freed immediately after cancel
request.

[client request thread]      [http thread]
1. download start
                           2. network doesn't work temporary
                           3. wait for auto retry (pthread_cond_timedwait(cond_http))
4. request cancel
                           5. pthread_cond_signal(cond_http)
                           6. free da_info
7. access to da_info for
   debugging message
   (fault)

Change-Id: I3bf0002e643af77dac17f69c543d29570c1f3cde

Wait for download thread before destroying da_info

A cancel request doesn't take effect immediately. (__http_gotheaders_cb)
So, below scenario can occur.

1. client-request thread requests cancel a download for a number of reasons. (ex. zombie client)
2. client-request thread is terminated.
3. queue-manager deallocates da_info_list for terminating.
4. download is canceled and then a http thread accesses to da_info for notifying download terminated.
5. heap-use-after-free occurs.

Change-Id: I47a7ef4c4d7ddda5f4594ee30885e3af230c6116

Fix invalid data type of parameter

Change-Id: I0e64b2fea19ca48dac78e53c2ffa3cf2bbc6106b

Free proxy_uri

In condition that proxy_uri is "0.0.0.0", proxy_uri is not freed.

Change-Id: Ifd3759c1a7b416cd69fc134eb653b0a5802a7a73

Return error when status code isn't 2XX or 3XX

Change-Id: Ie2a85db0c87060f9f275a0f86b075eefde284136

SAM: remove Global Variable violations

Change-Id: Ib3b4b47aa62517df80ea446ba405fd3f5caa5d33

Do not unload agent symbol file

If symbol file is unload before terminating program, segfault can occur
when access to static variable. (ex. mutex_da_info_list)

Change-Id: I9e6c61f2d45264682755d5717d85943e5b5c44a3

Revert "Do not unload agent symbol file"

This reverts commit 37808dc1789f8a3cdb99ce698ac38ec3a917a047.

Do not unload agent symbol file

If symbol file is unload before terminating program, segfault can occur
when access to static variable. (ex. mutex_da_info_list)

Change-Id: I9e6c61f2d45264682755d5717d85943e5b5c44a3

Merge changes I780f6179,I29223a72,I4e5182dd into tizen

* changes:
  Refine __dp_client_new()
  Add a function to handle init request
  Use additional functions in dp_client_manager()

Allow cancel request when download state is paused

download-provider cannot be terminated in case of below scenario.

1. application requests a pause.
2. download-provider pauses the download request.
3. application requests a cancel.
4. the download request couldn't be canceled because a state is paused.
5. http thread is alive constantly.

Change-Id: I7b2566bce98b3e563e2b5980c908a92140ffdcd9

Refine __dp_client_new()

Change-Id: I780f6179436819d471b4e99f2cc2387bc432d025
Signed-off-by: Cheoleun Moon <chleun.moon@samsung.com>

Add some functions to check request status

Change-Id: I76583efe22dd4ff7bd0d7151d8ab6f9de02bb3ae

Add a function to handle init request

Change-Id: I29223a721e50b0b06a19a329bfdcffaf436bbbbf
Signed-off-by: Cheoleun Moon <chleun.moon@samsung.com>

Use additional functions in dp_client_manager()

Change-Id: I4e5182ddae71600264cfe10e4ac69d88efa3c11d
Signed-off-by: Cheoleun Moon <chleun.moon@samsung.com>

Remove unnecessary predefined preprocessor

Change-Id: I376d831575e9c22e03691faa7ba0e3d76f682e63

Change HTTP state properly

Sometimes HTTP thread works abnormally because HTTP state isn't changed
properly.

Change-Id: I5462b6497bd5a7bd64c5b600a00aa7cf3e1954ff

tv: do not use thread locking function

libcurl has used openssl 1.1 since tizen 6.0.

Change-Id: Ie5b4db0bb7f3f85028373b985f02b3cff1c29db6

Deprecate pubkey-pinning

Pubkey pinning is not supported from tizen-6.0.

Change-Id: Ie3541d9780e70de55d2355e2e2544961a142aa80
Signed-off-by: Sangwan Kwon <sangwan.kwon@samsung.com>

Implement internal API for host verification

Change-Id: I0362a6263d177363580ecb95cf6223b611a1b896

Merge "Change CA certificate bundle and path for TV" into tizen

Accept legacy app path to keep backwards compatibility

Change-Id: I6164dd62f8a2c0a55732539a0bd5a73eb772d051
Signed-off-by: Seonah Moon <seonah1.moon@samsung.com>

Change CA certificate bundle and path for TV

Change-Id: I2029019f151df781ed589646e0b9ed69077f80fa

Replace app_context API with app_info API for getting package ID

P200120-04045

Change-Id: Id21ffee103b37ec9374e0f5655e3fe73ea7557e2

[CID-1103253] Fix bad bit shift operation

Change-Id: I91a989c5e6cac89bd64b97bf8ef835c0ac55fdce

bug fix: add error handling for preventing duplicate file

Change-Id: If3a00019e8b6d6ad586c5e2d5c38b0989c0fa966

Fix build warnings for gcc9

Change-Id: I57f8c411c130d0b48b9979d5eb0eaa859f239335

Support proxy authentication

Change-Id: I66b65fe2417db65bb2b0973ca950de2abd7dfea8

Add ssl thread locking function for openssl 1.0

TV profile doesn't use openssl 1.1 yet.
So, this patch defines thread locking function for TV profile temporaly.
It will be removed or disabled after openssl 1.1 is applied to TV.

Change-Id: Ia7a029b898d85cd84552e0fffe0966e1e14dac8f

Reduce cyclomatic complexity #1

provider/download-provider-client.c: __dp_request_get_info()

Change-Id: I4332b1611c23e2c521ce23ef0f9822dd858b87c7

Fix potential thread cancel issue

Change-Id: Id6f37948bfa6b7c3dd61cf2fdc9a585b1f3ec35e

Migrate to openssl 1.1

Locking mechanism is not needed anymore

Change-Id: If11be650c3366084c7bd28da90b5463348b1c880

Merge "Remove download-provider.service form graphical.target.wants" into tizen

Remove unused file

Change-Id: Ic42ed980a559da1fe466392c3015111e5506cd1d

Remove download-provider.service form graphical.target.wants

Download-provider.service is enabled by socket activation called download-provider.socket.
So, we don't need to execute donwload-provider.service on booting time.

Change-Id: I75df0aa3275174bb053806fede64d1ec77ab9f21

Remove unnecessary log

Change-Id: I8c54fdbd37fe712c66a798adb4b5a02e5923f22c

Merge "Change FILE_OFFSET_BITS to 64 bits to support a large file" into tizen

Limit g_dp_sock size for buffer overflow

[WID-3498058] Buffer 'rset.__fds_bits' of size 32
accessed at dp_client_manager() can overflow,
since its index 'g_dp_sock / 32' can have value 50331680
that is out of range, as indicated by preceding conditional expression.

Change-Id: I6c9c51903faa1df6c207670ab4b63b01038bcb70

Change FILE_OFFSET_BITS to 64 bits to support a large file

Change-Id: I034da2578c1355a93758fc960431410c7d2edf80

Remove unnecessary settings on download-provider.socket

Change-Id: Ib12324afbd6ad4ed99025ce4005988674eb8eaea

Add build macro to avoid conflict with spin

Change-Id: I64918d5f22a4d475b56d056a7eb3a2ce20044929

Fix merge conflict

Change-Id: Ib014b55edb95b0eafde42b3e39d17ff5929bf1ce

Expose new API to set/get http proxy from apps.

Need an API in download provider to set http proxy from app side.
There was no API defined in download provider interface for setting proxy.
Added new API dp_interface_{set,get}_proxy()

Change-Id: Ie79d34cebd4c27f372c7e50a42a0a809f88ca165
Signed-off-by: Sowmya B <sowmya.b1@samsung.com>

Changing download provider notification as in Dowload Manager's notification.

Download provider notification does not display URL information.
Added changes to show URL information in notification.
Also added changes to handle EBUSY case while releasing the acquired lock
on mutex.

Change-Id: I27bfff0495fb7da65fc0fb5deb7bd63b79d7a1a5
Signed-off-by: Sowmya B <sowmya.b1@samsung.com>

Sync with spin git

Change-Id: Iee53b7172a0076ac80ed457704a01a2edd839ba5

[DF190411-00679] do not handle empty file in case of redirection

Each time content size is 0, empty file is created even if status code means redirection.
(https://review.tizen.org/gerrit/#/c/platform/framework/web/download-provider/+/188428/)

Change-Id: I01e2755e912ae32e1c43c6f182356c05702c3d2d

[WGID-372659] reassign fd after using close()

Change-Id: I45081480991bf144dac2a2945e3e898e0b0856fa

Remove unnecessary files

Change-Id: I6739a6797dd86383d23e1bfb79e3e949c55ed6ba

[CID-835380] Access shared data with holding a mutex lock

Change-Id: I4cf8077e4c31c6df5f994b890f389c681a9dabd8

[DF190403-00436] Allow shared directory as destination path

Change-Id: Iade2ccfb783fe1475aae2d5b78b4443a9d2ddf2f

Fix memory leak

Change-Id: I4c16e3d4db431e0b24359bb21694c56403fd1c36

Do not call pthread_cancel to terminate an event manager thread

Change-Id: I05d840b25ccd59f1bcc30408a86504bc28709f86

[Security vulnerability] Prevent null pointer dereference

Change-Id: I24e3eab7d754cd5c3794a5eff412270ada9b352d

Get temp path from download daemon

Even if app doesn't set temp path,
temp path is updated when download is started.

Change-Id: Ifd624581ca42dc1fac5c5aa6b96ea3874297e736

Convert relative path to absolute path

Even if relative path is passed from user,
absolute path will be delivered to daemon.

Change-Id: I7616eb27f728bda81de4af6bc97aa5ec3b642f5b

[Security vulnerability] Replace strncmp() with strcmp()

In case comparing whole string, use strcmp().

Change-Id: Iacb667cacece5eb8a964c8dc2219ff0d17908bf3

[Security vulnerability] Access to a symbolic link

Change-Id: I2207dbe4a72d87cd6665fd27dfc97a30b4617b49