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

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

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

Merge pull request #742 from haesik/svace_fix_160318

Fix svace error (160318) - use of vulnerable function

Merge pull request #739 from heejin-kim/hj_18

/iotivity/examples: Remove unused .cpp files

Fix svace error (160318) - use of vulnerable function
Use of vulnerable function 'strcpy' at iotjs_module_dns.c
it's changed to strncpy

IoT.js-DCO-1.0-Signed-off-by: Haesik, Jun haesik.jun@samsung.com

Fix static analysis defects in IoT.js

Merge pull request #738 from haesik/svace_fix

SVACE error fix (160314)
 - missing initial values on iotjs_main.c

SVACE error fix (160314)

IoT.js-DCO-1.0-Signed-off-by: Haesik, Jun haesik.jun@samsung.com

Merge pull request #737 from jeongchanKim/sys_types

Move typedef unsigned long __kernel_size_t out of ifdef condition

Merge pull request #736 from jeongchanKim/coding_rule

Fix coding rule error

Merge pull request #735 from an4967/Fix_svace

Fix svace error - fclose

Move typedef unsigned long __kernel_size_t out of ifdef condition

IoT.js uses this type, so remove iotivity ifdef condition

Fix coding rule error

./os/fs/smartfs/smartfs_utils.c:3429: ERROR: [SPC_M_OPR] space prohibited before that ':' (ctx:WxE)
./os/fs/smartfs/smartfs_utils.c:3433: ERROR: [SPC_M_OPR] space prohibited before that ':' (ctx:WxE)
./os/arch/arm/src/s5j/s5j_boot.c:113: ERROR: [SPC_M_OPR] spaces required around that ':' (ctx:ExV)
./apps/examples/testcase/le_tc/kernel/tc_libc_math.c:1931: ERROR: [SPC_M_OPR] space required after that ',' (ctx:VxV)
./os/drivers/wireless/scsc/misc/utils_misc.h:255: ERROR: [SPC_M_OPR] space required after that ';' (ctx:VxV)

Fix svace error.

st_things_request_handler.c
- fix misspell in null check

easysetup_manager.c
- remove unreachable expression
- check negative value before close fd

things_security_manager.c
- move 'fclose(fp)' to general location

Merge pull request #734 from an4967/Fix_svace_errors

Fix svace errors

Fix svace errors

st_things_request_handler.c
- In error statement, make the func to return false

cloud_connector.c
- 'registerPayload' & 'loginoutPayload' are released in 'OCDoResource'
- So, add 'OCRepPayloadDestroy' to release in error state.

easysetup_manager.c
- Add check statement before close file descriptor

resource_handler.c
- change 'strncmp' which is using just for empty checking to 'strlen'

things_req_handler.c
- initialize variable before using

things_security_manager.c
- Add fclose after usage of fds.

things_string_utils.c
- Add '+1' when check string length for null character.

Merge pull request #733 from an4967/fix_svace_error

Fix svace error in st_things

Fix svace error

-  Add null check statement after allocating memory

Merge pull request #731 from jeongchanKim/fixedmath_doxygen

Fix doxygen comment in include/fixedmath.h

Fix doxygen comment in include/fixedmath.h

Remove @cond~@endcond comment and add @since comment for doxygen

Merge pull request #730 from jeongchanKim/lwip_dbg

Enable lwip debug configs and update the latest Kconfig in artik053/tc defconfig

Enable LWIP_ERROR and LWIP_ASSERT in artik053/tc defconfig

For debugging, enable lwip error and assert configs

Merge pull request #729 from heejin-kim/hj_18

Fix the st_things framework bug

Merge pull request #724 from chanijjani/fix_memoryleak_in_things_stack

fix memory leak in things_abort()

Merge pull request #728 from dr-venkman/fix_svace_warnings

Fixed SVACE warnings for IotBus, Lwip and iperf_api

Merge pull request #727 from jeongchanKim/iovec_redefine

Integrate iovec structure defined on uv-tizenrt.h and uio.h

Fixed SVACE warnings for IotBus, Lwip and iperf_api

Integrate iovec structure between uv-tizenrt.h and uio.h

There are same structure definition in different places. So integrated.

Merge pull request #718 from sangwon03/gpio_sync

fix i2c utc for read/write test

fix memory leak in things_abort()

Update utc_i2c.c

Merge pull request #723 from heejin-kim/hj_18

Fix the st_things example bug

Update utc_i2c.c

removed blank

Update utc_i2c.c

Merge pull request #687 from heejin-kim/hj_25

Add st_things example

Delete .project

Delete .cproject

Merge pull request #715 from junmin-kim/svace_lwip

Add NULL check in send_tcp function

Merge pull request #719 from jeongchanKim/spawn_api

Add doxygen comment for new kernel api : posix_spawn_file_actions_add…

Merge pull request #683 from heejin-kim/hj_18

Add st_things framework for SmartThings Things SDK

Add doxygen comment for new kernel api : posix_spawn_file_actions_adddup2

posix_spawn_file_actions_adddup2() is a new api for TizenRT v1.1

fix i2c utc for read/write test

I used the gy30 sensor for i2c test.
gy30 senser adress is 0x23 and it use just 1 byte packet for communication.
So I modified the adress and pakcet size.
And also I modified checking logic the result of read/write function because the return type of read/write is size.

Merge pull request #714 from shivgarg/svace

[Arastorage] Svace warning fix

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

This patch adds test cases for few API's namely:
dprintf, lib_rdflush, lib_snoflush, lib_sprintf, sprintf, ungetc.

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

Merge pull request #690 from heejin-kim/hj_252

Add st_things build configuration

Merge pull request #716 from junmin-kim/svace_s5j

Fix i2c return value when it fails to read

Fix i2c return value when it fails to read

Assign I/O error when i2c fails to read bytes,
and jump to fail laber with -EIO return value.

Signed-off-by: Jihoon Park <jh6186.park@samsung.com>

Add NULL check in send_tcp function

Pointer 'conn' which was dereferenced at api_msg.c:330 is compared to NULL value at api_msg.c:336.
If conn is NULL, ERR_ARG will be returned.

Corrected num of bytes being passed to memset in index_manager.c

Merge pull request #710 from Taejun-Kwon/audio

Add nuttx license in tinyalsa.c

Merge pull request #706 from an4967/add_build_config

Add build_config in Travis CI