Merge pull request #817 from HONGCHAEHEE/sdk

Fix assert during things reset

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.

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.

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

[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 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

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.

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()

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

apps/testcase/arastorage: refactor

1. add "static" for local functions
2. align definitions

apps/testcase/arastorage: remove registering of TASH command

To run arastorage testcase, TASH is not needed. It can be run by itself.

Merge pull request #785 from junmin-kim/artik053_download

Return exit code when openocd command failed in download script

Return exit code when openocd command failed in download script

If openocd return non-zero exit code, download script exits with exit code 1

Merge pull request #780 from jeongarmy/fix_doxygen

Update doxygen in arastorage.h

Add header file information to doxygen in arastorage.h

Add header file information to doxygen in arastorage.h
It helps user know that which header file should be included to use arastorage APIs

Modify doxygen comments in arastorage.h
- Add variable name to each param
- Fix typo and wrong description

arch/sidk_s5jt200: refactor s5jt200_led

1. A local variable is enough for led thread id.
2. Location of printing message is moved to give correct
   information of making led thread.

os: use kernel_thread instead of task_create in kernel side

The arch, logm and net belongs to kernel side so that those modules
should use kernel_thread instead of task_create.
The task_create is allocated at user side, the kernel_thread is at
kernel side.

Merge pull request #758 from olegartys/1015

logm/Kconfig: make LOGM dependent on !DISABLE_SIGNALS

Merge pull request #779 from junmin-kim/tc_defconfig

Update artik053 tc defconfig to enable net dup utc

Merge pull request #778 from jeongchanKim/modify_stkmon

Modify the stackmonitor : when CONFIG_DISABLE_SIGNALS is enabled, sta…

Modify the stackmonitor : when CONFIG_DISABLE_SIGNALS is enabled, stack monitor prints the stack info once

if CONFIG_DISABLE_SIGNALS is enabled, sleep() is not working.
becaus of that, stack monitor prints the stack info once and finished.

Merge pull request #752 from olegartys/1000

fs/smartfs/Kconfig: make SMARTFS_SECTOR_RECOVERY dependent on MTD_SMART

Update artik053 tc defconfig to enable net dup utc

This commit set TC_NET_DUP=y in artik053/tc/defconfig
Refer this net dup tc commit f9b8b822fcc5179e23c603eb11257ad5e298e367

Merge pull request #776 from an4967/add_iotjs_config_in_travis_ci

travis: Adjust build configs in Travis CI

travis: Adjust build configs in Travis CI

- Add artik053/iotjs build config in Travis CI.
- Remove duplicated test configs in Travis CI.

Merge pull request #767 from dr-venkman/fix_build_error_dm_utc_itc

Fixed errors when building for both DM utc and itc

Merge pull request #765 from an4967/fix_iotbus_pwm

pwm: fix issues on range and decimal point

pwm: fix issues on range and decimal point

- the range of pulse width should be 0 - 65535
- The max value should be b16ONE - 1 and the half value should be b16HALF.
- The rounding conversion satisfies above 2 condtions.

Merge pull request #771 from jeongchanKim/doxy_posix

Modify doxygen comment for POSIX API in include/sys and include/tinyara

Merge pull request #774 from junmin-kim/artik053_openocd

Set romfs_partition_enable using command instead of romfs_flashing.cfg

Merge pull request #768 from pillip8282/net_utc

net:tc add dup() testcase

Merge pull request #756 from olegartys/1013

apps/shell: hide usleep call if signals are disabled

Merge pull request #749 from an4967/st_things_build_config

travis: add new build config and install new tools in docker

Set romfs_partition_enable using command instead of romfs_flashing.cfg

partition_map.cfg did source romfs_flashing.cfg
But romfs_flashing.cfg is removed after "make download"
Sometimes cfg file are used without "make download"
So set romfs_partition_enable using openocd command
And romfs_partition_enable will be 0 if romfs_partition_enable doesn't exist

Modify doxygen comment for POSIX API in include/sys and include/tinyara

replace POSIX APIs -> POSIX API : the description is only for the below API, so the singular is correct

net:tc add dup() testcase

travis: add new build config and install new tools in docker

1. Add st_things build configuration
2.  Install cmake and scons

Fixed errors when building for both DM utc and itc

Callback functions for linkupEvent had multiple definitions for utc and itc.
These functions were redefined as static.

Merge pull request #769 from glistening/master

Fix IoT.js fs.scandir

Merge pull request #770 from jeongchanKim/doxygen_posix

Modify doxygen comment for POSIX API and SYSTEM CALL API

Modify doxygen comment for POSIX API and SYSTEM CALL API

1. replace POSIX APIs -> POSIX API
2. remove unnecessary square bracket

Fix IoT.js fs.scandir

Incorrectly located error handling code (introduced by #744) is fixed.
(The allocation error check should be after the allocation.)

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

* SCSC drivers implementation uses wpa_supplicant routines, so
  need to be dependent on WPA_SUPPLICANT

* SCSC driver implementation uses pthreads API, so their support
  must be provided, otherwise we are not able to compile driver.

Signed-off-by: Oleg Lyovin <o.lyovin@partner.samsung.com>

logm/Kconfig: make LOGM dependent on !DISABLE_SIGNALS

* logm_task uses usleep() call, which implementation
  relies on signals. So when they are disabled, usleep()
  routine is not compiled.

Signed-off-by: Oleg Lyovin <o.lyovin@partner.samsung.com>

apps/shell: hide usleep call if signals are disabled

* usleep() is not available if signals are disabled.

Signed-off-by: Oleg Lyovin <o.lyovin@partner.samsung.com>

fs/smartfs/Kconfig: make SMARTFS_SECTOR_RECOVERY dependent on MTD_SMART

* Implementation of sector recovery functionality requires mtd/smart.c
  routines, which compilation controlled by MTD_SMART.

Signed-off-by: Oleg Lyovin <o.lyovin@partner.samsung.com>

Merge pull request #748 from HONGCHAEHEE/wifi-temp

Quick fix for hang-up issue on Wi-Fi mode change

Remove the mutex use in wifi_linkdown_event_func

- Quix fix for hanging up during wifi mode change
- Remove mutex lock/unlock to prevent lock up

Merge pull request #746 from jeongarmy/fix_svace

Fix svace issue in st_things

Fix svace issue in st_things

- cloud_manager.c
 . Expression 'ES_STATE_INIT <= es_state' is always true , which may be caused by a logical error

- things_data_manager.c
 . Return value of a function 'cJSON_GetObjectItem' is dereferenced at things_data_manager.c:991 without checking, but it is usually checked for this function (79/83).

Merge pull request #744 from glistening/master

Fix SVACE defect in IoT.js (fs.scandir)

Merge pull request #745 from hs0225/master

IoT.js: Fix static analysis defects

Merge pull request #743 from heejin-kim/hj_25

Fix the svace error in /framework/st_things folder

IoT.js: Fix static analysis defects

Fix SVACE defect in IoT.js (fs.scandir)

Merge pull request #682 from thapav/stdio

Codecoverage(libc/stdio): Add TC's for few API's in stdio

Fix the svace error in /framework/st_things folder

Fix the dereferenced pointers and check null termination when using strncat and strncpy.

Merge pull request #740 from glistening/master

Fix memory leak on iotjs_module dns and iotjs_module tcp

Fix memory leak on iotjs_module dns and iotjs_module tcp

Fix memory leak reported by SVACE.

Merge pull request #741 from daeyeon/fix.svace

Fix static analysis defects in IoT.js