Fix a http header parsing function in webserver

- Remove the code for checking the length of the http header. The whole unseparated http header can be longer than the max key length.
- Re-fix of commit eda61d6f7b0b1ed4a17a62206ddf94d6fc8069d1

Merge pull request #803 from dr-venkman/fix_coap_parsing

Fix er-coap parsing in wakaama to handle heap buffer overflows

Merge pull request #812 from dr-venkman/fix_dos_issues_wakaama

Merge security-related fixes from eclipse wakaama Github to TizenRT

Merge pull request #827 from HONGCHAEHEE/doc

doxygen : Add include path and missing param name in framework

doxygen : tidy up framework/tinyalsa and st_things

* Add header include path for framework/tinyalsa and framework/st_things on doxygen comments
 - if not, we cannot know which header we should include for using that api

* modify indentations

Merge pull request #841 from jeongchanKim/doxy_include

doxygen : Add include path, brief desc and missing param name and fix…

Merge pull request #844 from heejin-kim/hj_18

Add header file info in iotbus API's doxygen comment

Add header file info in iotbus API's doxygen comment

You can see which path should be included to use iotbus APIs.

Merge pull request #811 from pillip8282/fix_netlib

net:netlib fix vulerability while parsing malicious URLs

Merge pull request #810 from pillip8282/fix_mdns

protocol:mdns fix vulnerability at mdns

doxygen : Add include path, brief desc and missing param name and fix wrong desc in os/include

1. Add include path for each apis
  - if not, we cannot know which header we should include for using that api
2. Add missing param name for each apis
  - argument name is needed after param[in] or param[out]
3. Add brief description for POSIX APIs
4. Fix wrong desc
  - some apis are not POSIX apis, but marked POSIX. so fix.

assert.h, crc16.h, crc32.h, crc8.h, debug.h, dirent.h, errno.h, fcntl.h, fixedmath.h, inttypes.h,
libgen.h, mqueue.h, pthread.h, queue.h, sched.h, semaphore.h, signal.h, spawn.h, stdio.h, stdlib.h,
string.h, syslog.h, termios.h, time.h, unistd.h
sys/prctl.h, sys/stat.h, sys/time.h, sys/wait.h, sys/ioctl.h
tinyara/clock.h, tinyara/math.h, tinyara/regex.h, tinyara/sched.h, tinyara/spawn.h, tinyara/streams.h,
tinyara/time.h, tinyara/ttrace.h

Merge pull request #839 from jeongchanKim/mqtt_codingrule

testcase/mqtt_itc : fix coding rule error - space is needed before '='

protocol:mdns fix vulnerability at mdns

1) fix overflow when mdns is parsing the packet
2) checking cycle reference when it is parsing the packet
    https://www.kb.cert.org/vuls/id/23495
3) adding a buffer overflow check when it is converting a name label

testcase/mqtt_itc : fix coding rule error - space is needed before '='

[SPC_M_OPR] spaces required around that '=' (ctx:VxW)

Merge pull request #832 from an4967/bug_fix_iotbus_pwm

iotbus: bug fix in pwm

net:netlib fix vulerability while parsing malicious URLs

netlib_parsehttpurl() should be returned when error is detected

Merge pull request #830 from an4967/apply_coding_rule

st_things: apply tizenrt coding rule

doxygen : Add include path and missing param name for mqtt APIs

1.Add include path for each api
•if not, we cannot know which header we should include for using that api
2.Add missing param name for each api
•argument name is needed after param[in] or param[out]

doxygen : Add include path and missing param names for DM APIs

1.Add include path for each api
•if not, we cannot know which header we should include for using that api
2.Add missing param name for each api
•argument name is needed after param[in] or param[out]

Merge security-related fixes from eclipse wakaama Github to TizenRT

Wakaama in the version currently used by TizenRT (seems to be from around March/April) is vulnerable to several security issues, which were since fixed in the upstream version at https://github.com/eclipse/wakaama.

Reference to the original fixes are given below:

1. https://github.com/eclipse/wakaama/issues/319 (fix unknown, but likely https://github.com/eclipse/wakaama/commit/8037174913a5937c3bcb5ffe7838e4d3b867c53e)
2. https://github.com/eclipse/wakaama/issues/320
3. https://github.com/eclipse/wakaama/issues/268
4. https://github.com/eclipse/wakaama/issues/301

Merge pull request #833 from shivgarg/audio

framework/tinyalsa : Removing pcm_set_config as public API

framework/tinyalsa : Removing pcm_set_config as public API

pcm_set_config has been removed from tinyalsa include file. The tcs in utc and itc which tested pcm_set_config have been removed. Calls to pcm_set_config outside of tinyalsa have also been removed .pcm_set_config has been made a static function so that it can it is called only from tinyalsa.c file.

iotbus: bug fix in pwm

- change type of divisor to get the result as float value.

st_things: apply tizenrt coding rule

- apply tizenrt coding rule

Merge pull request #828 from shivgarg/audio

ta_tc/audio/{utc,itc} : Removing assert failure in audio TCs

Removing assert failure in audio TCs

In audio utc/itc, readi positive tc gave an assert failure. The failure was due to the last pcm_readi call returning a negative value. The negative value was passed to write function which led to assert failure.
Now an if block has been added which checks the return value of pcm_readi before passing it to the write function.

Merge pull request #820 from pillip8282/fix_xmlrpc

external:xmlrpc handle an exceptional case when invalid parameters ar…

external:xmlrpc handle an exceptional case when invalid parameters are inserted

<> description:
Properly crafted xml request, can cause a situation in which the value of
the len field in the parsebuf_s structure will be less than the index field.
In this situation, when calling the xmlrpc_getelement from the xmlrpc_parseparams
function, the while loop will read the content of memory outside the buffer causing segmentation fault.

<> solution:
While calling xmlrpc_getelement function, check if index value inside parsebuf_s structure is less than len value.

Merge pull request #822 from arvinmittal/fix_systemio_i2c_itc

apps/examples/testcase/ta_tc/systemio/itc : fix itc_iotbus_i2c_write_read_p

Merge pull request #804 from arvinmittal/fix_systemio_gpio_itc

Fix systemio itc_gpio.c file, change gpio pin 12 to 41

Merge pull request #726 from arvinmittal/fix_audio_fw_itc

apps/examples/testcase/ta_tc/audio/itc: Add ITCs for Audio_Framework Module

Merge pull request #809 from thapav/fixAPIerrors

(libc/stdio): Fix wrong initialization in FS TC.

apps/examples/testcase/ta_tc/audio/itc: Add ITCs for Audio_Framework

Signed-off-by: Arvin Mittal <arvin.mittal@samsung.com>

Merge pull request #814 from pillip8282/wifimanager_network_event_signal

net:wifimanager fix sending net events to iotivity from wifi manager

Merge pull request #725 from arvinmittal/fix_mqtt_itc

apps/examples/testcase/ta_tc/mqtt/itc : Add ITCs for MQTT Module

Merge pull request #796 from an4967/mbedtls_hw_support

mbedtls: implement hardware support in mbedtls

apps/examples/testcase/ta_tc/mqtt/itc: Add ITCs for MQTT

Signed-off-by: Arvin Mittal <arvin.mittal@samsung.com>

Merge pull request #821 from jeongarmy/ttrace_config

testcase/ttrace: add select condition for using ttrace in Kconfig

testcase/ttrace: add select condition for using ttrace in Kconfig

Add select config in Kconfig for testcase of T-trace.
Enable CONFIG_TTRACE when EXAMPLES_TESTCASE_TTRACE is turned on.

Merge pull request #817 from HONGCHAEHEE/sdk

Fix assert during things reset

apps/examples/testcase/ta_tc/systemio/itc : fix itc_iotbus_i2c_write_read_p

Signed-off-by: Arvin Mittal <arvin.mittal@samsung.com>

Fix assert during things reset
- Assert often occur during the reset process as the TCP receive thread is trying to use resources of the ping thread already terminated.
- The TCP receive thread does not need to call the session state callback to terminate the ping thread during reset because the reset loop thread will unregister that callback.

(libc/stdio): Fix wrong initialization in FS TC.

This patch fixes wrong initialization in setvbuf and remove API testcases.

Signed-off-by: Vidisha Thapa <thapa.v@samsung.com>

Merge pull request #818 from jeongarmy/fix_tc_arastorage

testcase/arastorage: delete checking result value in cleanup function.

testcase/arastorage: delete checking result value in cleanup function.

cleanup function is called before and after arastorage TCs run to verify cleaning related resources
We don't need to check whether result value of function used here is valid or not.
Because this function is not tc and is used internally.

net:wifimanager fix sending net events to iotivity from wifi manager

Send network events to Iotivity only when the IP address is changed on the device.
Wifi manager sends network event even though sta leaves on SoftAP mode
In above case, The IP address on the device is still ok, so it doesn't need to send.

Merge pull request #816 from an4967/adjust_things_sample

st_things: update st_things sample app

Merge pull request #805 from sunghan-chang/tc

apps/testcase: refactor codes

st_things: update st_things sample app

- Add 2 dimmer resources
- Add handlers for additional resources

Merge pull request #808 from davidfather/wifi_manager_refactoring

Wi-Fi Manager: add a checkpoint, to check if Wi-Fi Manager is initial…

Merge pull request #797 from JeonginKim/gpio2

removed unsupported gpio output mode

Merge pull request #807 from jeongarmy/fix_tc_arastorage

testcase/arastorage: call cleanup function at the beginning arastorage TCs to verify cleaning all undeleted resources

Merge pull request #813 from jeongchanKim/coding_rule

testcase/filesystem : fix coding rule error - remove unnecessary space

Wi-Fi Manager: add a checkpoint, to check if Wi-Fi Manager is initialized, in APIs and callback handlers

To make Wi-Fi Manager thread-safe, it is necessary to check if Wi-Fi Manager is initialized.
For simplicity, we just do null-comparison of w_mutex and w_info_mutex at the very beginning of each API and each callback handler.

Merge pull request #750 from JeongJunSik/temp

st_things : Get the manufacturerName from JSON

st_things : add logic of parsing manufacturerName from JSON
- Changed the pre-defined value to use JSON value

testcase/filesystem : fix coding rule error - remove unnecessary space

[IDT_M_TAB] please, no space before tabs

remove unsupported gpio output mode

artik053 board is not support 2mode(IOTBUS_GPIO_DRIVE_NONE, IOTBUS_GPIO_DRIVE_PUSHPULL)

Merge pull request #720 from arvinmittal/fix_wifi_manager_itc

Add ITCs for Wifi_manager Module

Merge pull request #801 from JeonginKim/sec2

Fixed stack based memory disclosure in FTP client

Merge pull request #800 from an4967/remove_unused_ca

st_things: modify things_sdk security resources process

Fix er-coap parsing in wakaama to handle heap buffer overflows

Two cases of heap buffer overflows are considered:

1. When packet length is less than the coap header length of 4 bytes
2. when the option byte lengths have values far greater than what the packet buffer capacity. In this case, especially when merging contiguous location query coap options, buffer overflow tends to occur.

Functions modified: coap_parse_message, and coap_merge_multi_option.

[TizenRT][Added ITCs for Wifi_manager]

Signed-off-by: Arvin Mittal <arvin.mittal@samsung.com>

Merge pull request #795 from HONGCHAEHEE/web

Fix stack buffer overflow in webserver

testcase/arastorage: call cleanup function at the beginning arastorage TCs to verify cleaning all undeleted resources

cleanup function cleans all comopents like relations used in these TCs.
Some files used before in arastorage operations and so on may be removed.
So we call cleanup function before running arastorage TCs to verify cleaning all existent things.

apps/testcase: refactor codes

1. remove unused definition, RETURN_ERR
2. remove unused header file, tc_internal.h on systemio itc
3. fix incorrect definition of preventing duplicated-including header file
4. move a location of including header to inside definition
5. remove unnecessary relative path of header, tc_common.h
6. remove definition of same global variables and initialize them before using,
  total_pass and total_fail

Fix systemio itc_gpio.c file, change gpio pin 12 to 41

Signed-off-by: Arvin Mittal <arvin.mittal@samsung.com>

Fix stack buffer overflow in webserver
- Add the code to check that the resulting string is null-terminated.
- Add the code for url length to respect the buffer sizes.

Merge pull request #791 from JeonginKim/master

Memory corruption security issue fixed for FTP client

mbedtls: apply coding rule and adjust Kconfig in mbedtls

- apply tizenrt coding rule
- adjust Kconfig in mbedtls

Fix stack based memory disclosure in FTP client

Function ftp_pasvmode that parses FTP server response to PASV command goes outside data buffer - ptr is increased and checked only using !isdigit function.

tls: fix memory leak

If you do not release the allocated memory and return it, a memory leak
may occur.

Change-Id: I27a892c56e8ca381679d730c95014455b5de19b9
Signed-off-by: Junhwan Park <junhwan.park@samsung.com>

Merge pull request #799 from jeongarmy/modify_tcname

testcase/fs : change the name of all filesystem TCs to tc_[module]_[function]

st_things: modify things_sdk security resources process

- remove unused CAs
- check svrdb file contents before initialize st_things

testcase/fs : change the name of all filesystem TCs to tc_[module]_[function]

The name of all testcases in le_tc has naming rule, tc_[module]_[function]
So I need to change all testcases for fs, stdio in fs_main.c to apply it

Merge pull request #759 from olegartys/1017

drivers/scsc/Kconfig: make SCSC_WLAN dependent on WPA_SUPPLICANT and …

Merge pull request #793 from chanijjani/fix/delete_dead_code

Delete dead code related to things_data_manager

Merge pull request #783 from HONGCHAEHEE/wifi-mode

Fix the WiFi linkdown hang up issue

Merge pull request #794 from jeongchanKim/svace_env

testcase/environ : check null case when getenv()

sss: change ispdriver library for sss and add header files

This library adds features to improve performance when tls is performed.

Change-Id: Ifb344de9a0cead5c22eee119557db0a140802105
Signed-off-by: Joohwan Kim <joohwani.kim@samsung.com>

net/tls: apply mbedtls patch to prevent bypass of authentication issue

If optional authentication is configured, allows remote attackers
to bypass peer authentication via an X.509 certificate chain with
many intermediates. So, this commit applies patch to prevent thus issue.

Change-Id: If77de1f62928c1107e420c2b41bccf6186f19599
Signed-off-by: Joohwan Kim <joohwani.kim@samsung.com>

net/tls: add see_misc file for supporting miscellaneous functions

This file includes feature for getting uuid of ARTIK
certificate.

Change-Id: I6e9a65293edbdf2dd618f2faad4092ffa80d37bd
Signed-off-by: Joohwan Kim <joohwani.kim@samsung.com>

net/tls: change HW_ECDSA_VERIFICATION config location

Because see_setup_key_internal function is common api, it doesn't
need to be bind by HW_ECDSA_VERIFICATION config.

Change-Id: I6e7cd49fb5d2988619e4a5199b44deac0bae77c1
Signed-off-by: Joohwan Kim <joohwani.kim@samsung.com>

net/tls: add sss hw interface parts in mbedtls

Add the following in mbedtls
- performance : merge see_internal functions
- tls with secure storage : merge see_api functions

Change-Id: I0126bddcc83fe801a52ee66fea200d4b1c4cf8a3
Signed-off-by: Joohwan Kim <joohwani.kim@samsung.com>

net/tls: add see apis for security

This commit adds see apis for security full features.
This is for hardware acceleration and data/key security.

Change-Id: I62ab8ba77da0953bfa0f895679a509a9bfb17ce1
Signed-off-by: Joohwan Kim <joohwani.kim@samsung.com>

Fix the WiFi linkdown hang up when changing the WiFi manager mode
- Hung up during linkdown because the linkdown handler did not consider WIFI_MODE_CHANGING state.
- Add wifi manager modes for linkup&down handlers to know previous and next modes.
- Edit functions of wifi init, deinit and set mode which trigger linkup&down handlers.
- Re-fix for the previous quick fix(commit 111d0b10430601ea4653b2be3530b94dc9848ffd)

Merge remote-tracking branch 'upstream/master'

testcase/environ : check null case when getenv()

Return value of a function 'getenv' is dereferenced at tc_environ.c:159 without checking, but it is usually checked for this function (14/17).

s5j/clk: modify SPI clk rate

- An SPI contains an internal 1/2 clock divider. Configure the SCLK_SPI
  value to the double of an SPI operating clock frequency.

- When you check the source without the commit log, it can be difficult to
  understand why you doubled it. I added a comment.

Change-Id: I8efd88dec2166ae5241f13a08d2c18438d26bb9b
Signed-off-by: Junhwan Park <junhwan.park@samsung.com>

build: add starter kit pinout figure

If you have an ARTIK 0 Starter Kit board, you can refer to PinOut with
this document.

Change-Id: Iaee1b9ebfab25c8def33e215285847d2716bfa62
Signed-off-by: Junhwan Park <junhwan.park@samsung.com>

Delete dead code related to things_data_manager

Merge pull request #772 from pillip8282/fix_coap

protocol:libcoap Fix build error

Merge pull request #788 from JunhwanPark/adc_log

s5j/adc: change interrupt debug level

Merge pull request #787 from JunhwanPark/iotbus_uart

iotbus/uart: add missing break

Merge pull request #781 from sunghan-chang/kthread

use kernel_thread for kernel thread and refactor s5jt200 led

Memory corruption security issue fixed for FTP client

protocol:libcoap Fix build error

mbedtls and libcoap cause a conflict because there are object files which are same name
To fix it I added prefix at object files

s5j/adc: change interrupt debug level

An unnecessary warning log was output. The debug level was modified to
prevent output by default.

Change-Id: If5a70358171450338699997de5a2b070c92566d5
Signed-off-by: Junhwan Park <junhwan.park@samsung.com>

iotbus/uart: add missing break

A break statement was missing inside switch case. no break statement,
hence stopbits 2 always failed.

Change-Id: Ib2b53ef73e752821f1b077700606fcb45d403597
Signed-off-by: Junhwan Park <junhwan.park@samsung.com>

Merge pull request #786 from sunghan-chang/tc

modify arastorage testcase