Revert "Revert "Revert "Modify the SMACK label for SDB shell."""

This reverts commit cb4d1bb8af186a59661e4e35bdb13fecdc1a0163.

Change-Id: I0e22b3fe0cb56f38cc562182bef7a65de998a59f

Revert "remove smack_setlabel function usage for security reason"

This reverts commit 3b551c517915ee6b2c4709a57dc066ea64c29973.

Change-Id: Ie6f76b81f12a736ac797ccb882ff7b922c0b621e

Revert "remove mkdir for sdbd log file"

This reverts commit 7d1c2eecd0f6ab44be3d4d4d1d9634af9b0a3aa7.

Change-Id: I59138c8c1b068100e45eb5d0511934cdab7f0a4a

Revert "remove mkdir again for sdbd log file"

This reverts commit ec96660d649e7eb961046f6f456eade9a0ea6920.

Change-Id: I8ed9f39eea79ab0a96dee071e491387843e84345

remove mkdir again for sdbd log file

Change-Id: I3ca69e6cce164c1df13dcbee6c453e17d3bf60da
Signed-off-by: SangJin Kim <sangjin3.kim@samsung.com>

remove mkdir for sdbd log file

remove mkdir for sdbd log file

Change-Id: I50a7902cd1738a687bc8c315998b29a33a8b720f
Signed-off-by: greatim <jaewon81.lim@samsung.com>

Merge "remove smack_setlabel function usage for security reason" into tizen_3.0

remove smack_setlabel function usage for security reason

remove smack_setlabel function usage
change sdbd log directory
change sdbd smack label to "System"

Change-Id: I242c08d177f456768f3b6e3d3ee90bbb70d7dbe9
Signed-off-by: greatim <jaewon81.lim@samsung.com>

Merge "Revert "Revert "Modify the SMACK label for SDB shell.""" into tizen_3.0

Revert "Revert "Modify the SMACK label for SDB shell.""

This reverts commit 37ca0ed4a0610cc20f954cb94a2ac7698ba52d56.

Change-Id: Idc727cc0a259d750634d3ef70b4f71dc9a160eba

Fix SVACE issue.

Change-Id: I640624dc49117b8eca6034b41ae33b2e52b8eb8c
Signed-off-by: SangJin Kim <sangjin3.kim@samsung.com>

fix potential bugs

fix NO_LOCK.STAT bugs for errno

Change-Id: Ic24d17fdf755d9d7b007db68b50b6fb2b30cee37
Signed-off-by: greatim <jaewon81.lim@samsung.com>

Merge "Revert "Revert "Modify the packet size in transport protocol.""" into tizen_3.0

Revert "Revert "Modify the packet size in transport protocol.""

This reverts commit 57917a169d72ead67de0921c6f1a37fe70f99040.

Change-Id: I37a23f2cea79831b3c532c8adba3bfce0b95d05d

fix potential bugs

remove unnecessary files and code (properties.h and properties.c)
fix double close problem of socket pair in file_sync_service.c
remove unnecessary manifest file

Change-Id: Ic61ca8ffab9e1ee31f3a432acf0e9c138fc26fbe
Signed-off-by: greatim <jaewon81.lim@samsung.com>

Merge "Revert "Modify the packet size in transport protocol."" into tizen_3.0

Merge "Revert "Modify the SMACK label for SDB shell."" into tizen_3.0

Revert "Modify the SMACK label for SDB shell."

This reverts commit 046f23e7ea9c15c72de74fc46530c3319bc7622d.

Change-Id: I196d5d4d975e9aec1efd28a39c3f1aaf2a9da6f9

Revert "Modify the packet size in transport protocol."

This reverts commit f7a9e14ff28fdb099abbfba917a2d135e354c080.

Change-Id: I1b6f418aaec7ce07b30e98989841cabb4f01dc1e

Merge "Modify the SMACK label for SDB shell." into tizen_3.0

Modify the SMACK label for SDB shell.

- Previously, there was a problem that the SDB shell
  has the SMACK authority of sdbd.
  In order to prevent this, it has been modified to run
  a separate SMACK label between sdbd and SDB shell.

Change-Id: I768ef0b165cf66fe88648cc272638819c4bd96fe
Signed-off-by: Kim Gunsoo <gunsoo83.kim@samsung.com>

Merge "Modify the packet size in transport protocol." into tizen_3.0

Modify the packet size in transport protocol.

- increase the transport packet size. (4KB -> 256KB)
- read USB transport packet as 4KB block repeatedly

Change-Id: I949ff050e51b907523922104bd5683b1132f2b2c
Signed-off-by: Kim Gunsoo <gunsoo83.kim@samsung.com>

fix potential bugs

fix potential bugs
1. NO_LOCK.STAT for errno in log.h
2. NULL_AFTER_DEREF by asprintf

Change-Id: I3470fd2d8d0b86c3417add3657a38b45d1222626
Signed-off-by: greatim <jaewon81.lim@samsung.com>

disable printing log in emulator by default

disable printing log in emulator by default

Change-Id: I88ad00f7824b055847e0dcb8d580675334e12d7b
Signed-off-by: greatim <jaewon81.lim@samsung.com>

fix a bug (free the static buffer)

fix a bug (free the static buffer)

Change-Id: If31caa5c8ff36ae7658bb33daa1df07e20f053be
Signed-off-by: greatim <jaewon81.lim@samsung.com>

modify default app command behavior for uninstall command

modify default app command behavior for uninstall command

Change-Id: I8acbd3761af9672f6d51a2f94330bfe3509938da
Signed-off-by: greatim <jaewon81.lim@samsung.com>

fix a bug that cause dynamic overflow

fix a bug that cause dynamic overflow

Change-Id: I1cf2159915442a98257e06948bcb193886ccb547
Signed-off-by: greatim <jaewon81.lim@samsung.com>

refactor calling encryption functions of plugin

modify the implementation of encryption function according to new plugin architecture

Change-Id: I62c142b5bf95fe7c4d8529c9272213274883b0a5
Signed-off-by: greatim <jaewon81.lim@samsung.com>

fix to add null-termination character when copy string by strncpy

fix to add null-termination character when copy string by strncpy

Change-Id: Icad071a0f17a0f1e8aff893cefd343ba12e2dd66
Signed-off-by: greatim <jaewon81.lim@samsung.com>

Fix the bug that a shell command buffer is not initialized

Change-Id: Ib295d3c8a4782d71add779c659e9c4a99dda6304
Signed-off-by: SangJin Kim <sangjin3.kim@samsung.com>

Fixed SVACE issue

Fixed SVACE issue

Change-Id: Ia5f7ee4c56ae82702809ff0d69eb229594647a1f
Signed-off-by: greatim <jaewon81.lim@samsung.com>

fix a bug that sdb shell command over 128 byte is cut

modify string buffer size of parameter for plugin invocation

Change-Id: Ia3925eca3d444bf4b1c66c7c6c9827225ea90294
Signed-off-by: greatim <jaewon81.lim@samsung.com>

Revert "Revert "add data encryption feature""

This reverts commit e5cb6422d20125dc1896e7d8838c6e7e8b77ab86.

Change-Id: I9d806afef3826e7c7f40b613bd6b4dd06064b2e8

Revert "Revert "Merge plugin improvement commit""

This reverts commit 84bfce8dd67d9d74febaf5ef3c0fe0cd6ec16382.

Change-Id: I7d5ca566b67230ca712ec276f74ad2115b66e2ca

Revert "Revert "modify default plugin's behavior by emulator""

This reverts commit b94610b00ff7573e1f16c8f08cc92d11d88d080a.

Change-Id: I8d4d37517c8cd13a6a8e763a68a5f1ccd19038bd

Revert "Revert "Add the appcmd protocol for product extended routine.""

This reverts commit 49a0a10876e193945a71849f0519b19883f4752a.

Change-Id: If226fb2f2118877976170d80406b53f318344e89

Revert "Revert "improve plugin architecture""

This reverts commit eb26b877fe2db374ffa875bbb1e6c1cf19921c86.

Change-Id: Id98f5e6d21f4f96bc3ffb4ca6ae928ea07232b04

Revert "improve plugin architecture"

This reverts commit 4290e4be7477c1bfec81607cd17c3907c5b315c6.

Change-Id: I52177f71b9dfb5ac50c77c93b0a94feb203fef04

Revert "Add the appcmd protocol for product extended routine."

This reverts commit e30355d93562b2edc8ad596149ccf3aa7140a906.

Change-Id: I244670cbf42d885f1626db1a7fe2a51251e0eb61

Revert "modify default plugin's behavior by emulator"

This reverts commit a430ea553950d5a9910cc051c3171da852607c03.

Change-Id: I23c07c65ac71009a9e48a60b3e6e9f1f4586fe96

Revert "Merge plugin improvement commit"

This reverts commit 97f2d53e382e658536c6e38073f695812b57af38.

Change-Id: I71c2fbae3f8c3f439304ab51224fdb3daecbdb0e

Revert "add data encryption feature"

This reverts commit 89745858c0ca69210ece6e794d980fc7bfc51f96.

Change-Id: Id0fb6cab639e885c802fd956de1bdab820cc8d47

add data encryption feature

add data encryption feature
 - encryption on / off for each device
 - encryption message handshaking
 - encryption module

Change-Id: I89337af9d9a5bc6fac45401f962392491b3e43ae
Signed-off-by: greatim <jaewon81.lim@samsung.com>

Merge plugin improvement commit

Merge plugin improvement commit

Change-Id: Ic0e4f94dd9874a5cea4885a5fda7ff0de2441388
Signed-off-by: greatim <jaewon81.lim@samsung.com>

modify default plugin's behavior by emulator

modify default plugin's behavior by emulator
add more debugging log

Change-Id: I2d5fdd4c5a56f63113dc2d34622afcdc2eacb345
Signed-off-by: greatim <jaewon81.lim@samsung.com>

Add the appcmd protocol for product extended routine.

- In order to reduce the platform-dependent implementation,
  it is added to appcmd protocol that is a product extension structure.

Conflicts:
src/sdb.c

Change-Id: I5ad12f8f3db0afdb588dbcaf49f98c7390cb794f
Signed-off-by: Kim Gunsoo <gunsoo83.kim@samsung.com>

improve plugin architecture

add default_plugin_*.c file for default plugin code
add plugin.c/h file for plugin related code (plugin loading, unloading)
refactoring plugin related codes

Conflicts:
src/sdb.c
src/sdbd_plugin.h
src/transport_local.c

Change-Id: I3cc8858364f0121819c693713aa9772e4c633047
Signed-off-by: greatim <jaewon81.lim@samsung.com>

Modify the type reading CPU architecture type from sysinfo.

- To support 64bit CPU architecture, the type reading from
  sysinfo was changed to string.

Change-Id: I88ac5d7ad2e76d4c2a83cb2912fa2db49d3cc809
Signed-off-by: Kim Gunsoo <gunsoo83.kim@samsung.com>

Adds error handling when a change of user privilege has failed.

- When the authority to change the shell and sync service process
  has failed, add the error handling.

Change-Id: I78a5ee314cad9a881a16dc7817ab6c85e11f0d57
Signed-off-by: Kim Gunsoo <gunsoo83.kim@samsung.com>

modify spec file to make dev package

To support pkgconfig for sdbd plugin,
modify spec file to make dev package

Change-Id: Ic9095cab424273d1ca3c526a779f5a4c23556abb
Signed-off-by: greatim <jaewon81.lim@samsung.com>

Fixed SVACE issue.

- Remove the vulnerable function 'strerror()'.

Change-Id: I6f056b85a6ec55ab7221fd977e6da61558432904
Signed-off-by: Kim Gunsoo <gunsoo83.kim@samsung.com>

Merge "Revert "change umask value to 022 from 000 because of security reason"" into tizen

Revert "change umask value to 022 from 000 because of security reason"

This reverts commit f17729d36596c1bb347a3ade5d86f6d05bd931e3.

Change-Id: Id9368d876bf0c3cfe068d7e4567dff8b4bec135b

Merge "Fix the sdb shell malfunction in ASan build" into tizen

Merge "Fixed a buffer underflow which could happen during processing sdb shell command." into tizen

Merge "Add KEEP_ALIVE option to the socket connected with sdb server." into tizen

Merge "Move variables declarations from header to source" into tizen

Fix the sdb shell malfunction in ASan build

Accordingly to exec(3) manual the envp argument is an array of pointers to
null-terminated strings and must be terminated by a null pointer.

Without the terminator execvp fails and errno is set to 'Bad address'

Change-Id: I00e6927fd62323b7a9bea918cc79b0c4e02d77fd
Signed-off-by: Slava Barinov <v.barinov@samsung.com>

Add KEEP_ALIVE option to the socket connected with sdb server.

- After connected with sdb server through TCP socket, add KEEP_ALIVE
  option to the socket.

Change-Id: I16ff01721dd8f919782c666871053a6e7599ec0e
Signed-off-by: Kim Gunsoo <gunsoo83.kim@samsung.com>

Fixed execve EFAULT error on the asan environment.

- the envp array of pointers must be terminated by a NULL pointer.

Change-Id: Ieb64b20c1fdf24aec16ae248faf051889ddb76d4
Signed-off-by: Kim Gunsoo <gunsoo83.kim@samsung.com>

Fixed a buffer underflow which could happen during processing sdb shell command.

Change-Id: I0faa48855632d00e18b9e054b4f83e786b24bdd9
Signed-off-by: shingil.kang <shingil.kang@samsung.com>

Merge "change umask value to 022 from 000 because of security reason" into tizen

change umask value to 022 from 000 because of security reason

change umask value to 022 from 000 because of security reason
add S_IWOTH for pushed file to owner's content directory

Change-Id: Ie0677c4bfa8c494c13ab0ac32cdb730f90d22864
Signed-off-by: greatim <jaewon81.lim@samsung.com>

Add the can_launch member to capability structure.

- The can_launch member is used to determine whether to
  install/launch an Application for the corresponding platform
  in the IDE.
- To obtain the can_launch information, queries the sdbd plugin.

Change-Id: Ic39cc0dce2267190de2e6052172a6333a1e5882b
Signed-off-by: Kim Gunsoo <gunsoo83.kim@samsung.com>

Fixed the build problem for emulator 64 bit architecture.

- modified the way which determines device type (emulator or target)

Change-Id: I1742336fcf128649362d4b855777fa6312636b8d
Signed-off-by: shingil.kang <shingilkang@samsung.com>

Merge "Checks whether socket transport is duplicated." into tizen

Merge "Added 'environment_variables' capability" into tizen

Checks whether socket transport is duplicated.

- When connecting with sdb server, to prevent abnormal
  connection request the check routine is added.

Change-Id: Ifd59f87f826dba442aa5af08145a5dcab9fd61c8
Signed-off-by: Kim Gunsoo <gunsoo83.kim@samsung.com>

Merge "Removed that debug-launchpad is started or stopped by sdbd" into tizen

Added 'environment_variables' capability

- Environment variables are applied to sdb shell process environment.
- Each environment variable should be separated by new line character('\n').
  e.g.) A=B\nC=D
- The length of environment string should be less than 4096 byte.
- The total number of environment variables should be less than 100.

Change-Id: If5b52217b364ecf45eb88f0d8d3644cec40d3d30
Signed-off-by: shingil.kang <shingil.kang@samsung.com>

get valgrind binary path from platform configuration

get valgrind binary path from /etc/tizen-platform.conf so that valgrind terminates normally

Change-Id: I7f88c193cd32a7c347122ce053851eb04b64d1b4
Signed-off-by: woojin <woojin2.jung@samsung.com>

Removed that debug-launchpad is started or stopped by sdbd

Change-Id: Id6aa4c6d6fee7976b929c33d4b1b4ede0647dff1
Signed-off-by: shingil.kang <shingil.kang@samsung.com>

Set SmackProcessLabel to System::Privileged

Change-Id: I2420fc1d679607bd2d9b855ec9547a31ee98290e
Signed-off-by: Yunjin Lee <yunjin-.lee@samsung.com>

Fixed build warnings

Change-Id: I45ea4e8e5321e2d0ebe9dc1e6dc5d7ffe76b48f4
Signed-off-by: shingil.kang <shingil.kang@samsung.com>

Add PATH environment variable definition into script file

- To prevent security attack, add PATH environmeent variable into shell script file

Change-Id: I58610f40d1d3d8af5de0ad55b1a037e5ef99b02b
Signed-off-by: greatim <jaewon81.lim@samsung.com>

Deletes the files that have not been used.

- It erased the files of sdb server code and features
  that are not currently in use.

Change-Id: I626ccee4da7d98d85646d56f798dc089ccce2dd6
Signed-off-by: Kim Gunsoo <gunsoo83.kim@samsung.com>

Fixed codes which may occur potential bugs.

- check null pointer
- fix double close

Change-Id: I7baacb4d01d74a883ef969e43a14035ee89307a1
Signed-off-by: shingil.kang <shingil.kang@samsung.com>

Disabled OOM killing for SDBD process on TV profile

Change-Id: I721285019b58a7f2d1004bac25c52606aa3d3738
Signed-off-by: shingil.kang <shingil.kang@samsung.com>

Added capabilities ralated to sdbd log

- sdbd_log_enable : enable or disable sdbd log
- sdbd_log_path : the folder location of sdbd log

Change-Id: I52b91ba2242536474fe6b6f0e34dd48ddebc8a50
Signed-off-by: shingil.kang <shingil.kang@samsung.com>

Remove unnecessary smack label for running sdbd.

- The label(User) of the running sdbd no longer needed.

Change-Id: I6b4918d332e73d6db71a584c6a1d37906b8f6a8d
Signed-off-by: Kim Gunsoo <gunsoo83.kim@samsung.com>

Returns error code when it fails to get the value of tzplatform envrionment variable.

- return 0 when succeeded
- return 1 when error is occured while getting the value of tzplatform environment variable.
- return 2 when tzplatform environment variable is not valid.

Change-Id: Ie4530148c43d757f69a3e8457777cbf25cb4dc3d
Signed-off-by: shingil.kang <shingil.kang@samsung.com>

Merge "Added the new protocol to get the value of environment variable." into tizen

Merge "Get the path where crash file is generated by using TZ_SYS_CRASH environment variable." into tizen

Added the new protocol to get the value of environment variable.

- 'env:{environment_variable}' prototol is used to get the value of {environment_variable}

Change-Id: Ie96c2318665c45950c7c17d156da378b62b1ea1e
Signed-off-by: shingil.kang <shingil.kang@samsung.com>

Get the path where crash file is generated by using TZ_SYS_CRASH environment variable.

Change-Id: I954a722b617b90ed8741b6080ed7fa1d20365ee0
Signed-off-by: shingil.kang <shingil.kang@samsung.com>

Revert "Modify modules that need root permission."

This reverts commit c0cc4c69e65107bb2b0f955c2589ca53551c6940.

Change-Id: I669f55ba15c9757ef34e5c041b85585633d3a0e8

Merge "Revert "sdb: change group and owner of sdb device node"" into tizen

Revert "sdb: change group and owner of sdb device node"

This reverts commit 935e0b600c01ef6c81e22053453fe32e2c53995c.

Change-Id: I4d6b1fcf839d8b4e222a1c50f2267bb03b7f8e23

Move variables declarations from header to source

This prevents 'multiple definition' error when building with -fno-common this is
used in Address Sanitizer builds.

Change-Id: I0004622648c6e442bfed36f7aa14405b858d995b
Signed-off-by: Slava Barinov <v.barinov@samsung.com>

Merge "sdb: change group and owner of sdb device node" into tizen

Merge "Modify modules that need root permission." into tizen

sdb: change group and owner of sdb device node

- The owner and group of sdbd are changed from "root "to "sdk".
  Thus the udev rule is changed for it.

- Previous:
    bash-3.2# ls -alZ /dev/samsung_sdb
    crw------- 1 root root * 10, 15 Jan  1 23:41 /dev/samsung_sdb
    bash-3.2#

- Now:
    bash-3.2# ls -alZ /dev/samsung_sdb
    crw-rw---- 1 sdk sdk * 10, 15 Jan  2 04:09 /dev/samsung_sdb
    bash-3.2#

Change-Id: Ifef41987d391bc544cd8792fc21458624306ce29
Signed-off-by: Taeyoung Kim <ty317.kim@samsung.com>

Added sdbd service to emulator_preinit.target

- Sdbd needs to be booted early for debugging.
- Moved the target of sdbd service to emulator_preinit.target from emulator.target

Change-Id: I55708052dd8b831684a0a83139935cd14e96af98
Signed-off-by: shingil.kang <shingil.kang@samsung.com>

Fixed SVACE issue.

- Invalid function definition.

Change-Id: I9b3367185add114459359d25422985ab9599d4c7
Signed-off-by: Kim Gunsoo <gunsoo83.kim@samsung.com>

Modify modules that need root permission.

- Removed to permit root permission for push/pull(unitest, codecoverage, DA)
- Changed uid/gid of SDBD into sdk

Change-Id: Idd874377159afa3d387c9abb59aada2a9d34dbc3
Signed-off-by: shingil.kang <shingil.kang@samsung.com>

Change the routine for determining whether the emulator or device.

- Determined to look at kernel parameters that includes the vm_name
  in case of emulator.

Change-Id: I06734a63eb3afcb9b9638bc1044275e5291fc5eb
Signed-off-by: Kim Gunsoo <gunsoo83.kim@samsung.com>

Added to check validation of pointer

- If packet pointer has invalid address then restart SDBD.

Change-Id: Ieff9343bba35dcfe5122c18ddea60eec615bac76
Signed-off-by: shingil.kang <shingil.kang@samsung.com>

Merge "Modify the sdk groups routine for sdb shell/sync service." into tizen

Modify the sdk groups routine for sdb shell/sync service.

- Add the groups for sdbd shell/sync default group.
  ("priv_externalstorage", "priv_externalstorage_appdata", "priv_mediastorage")
- Modify the sdk group init routine that receives group infomation
  from platform.

Change-Id: I7273df2e05f65a62953e7e4f6003e3bc928aaae4
Signed-off-by: Kim Gunsoo <gunsoo83.kim@samsung.com>