subprocess: remove user shell

sub-process uses /sh instead of a copy of it.

Change-Id: I2f0eb13fe5f08ae4eb6b1f3e57fd6d4119d7e446
Signed-off-by: Jinhyung Choi <jinh0.choi@samsung.com>

package: update version (3.0.17)

Change-Id: I11e16a48bb2367ea7ef4c97a3e8649734185799f
Signed-off-by: Sooyoung Ha <yoosah.ha@samsung.com>

source: remove some risky codes

Some of these codes can cause the various defects.

Change-Id: I5effcdba397a9f80583564ccb2f975205413543c
Signed-off-by: Sooyoung Ha <yoosah.ha@samsung.com>

package: update version (3.0.16)

Change-Id: I49f60d84a2a061c71cd1a884af73306ad52ddd35
Signed-off-by: Sooyoung Ha <yoosah.ha@samsung.com>

shell: introduce sdb user daemon for user shell

To create a shell session via sdb, SDBD open the ptmx and then a pts
device is created. Next, SDBD forks a shell who use the pts opened
previously. BUT, forked shell cannot use the pts because forked shell
has no privilege to read the pts.
So I introduce new sdb user daemon who is only for launching shell.
The sdb user daemon has low privilege, same as user shell, and has few
functions - open ptmx, pass opened ptmx fd to SDBD, and exec shell.

Change-Id: I2332419e4b186d4b18c281a554569735617e9f64
Signed-off-by: Sooyoung Ha <yoosah.ha@samsung.com>

source: remove redundant code

Remove some duplicated codes and redundant spaces.

Change-Id: I4c42490f999eb587f7adca17b3322ca900af50c1
Signed-off-by: Sooyoung Ha <yoosah.ha@samsung.com>

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

This reverts commit 45bfe8e57fd16c8bc2ad8073afeb3a842c3eb810.

Change-Id: Iecfb11a43fb26135cc4f99260892d7edd1e69722
Signed-off-by: Sooyoung Ha <yoosah.ha@samsung.com>

Revert "Revert "Modify modules that need root permission.""

This reverts commit 9395230cf70cc07bfd036f8aa374f5f9443c7983.

Change-Id: I3fb22370706706e8e8042fedd58b79d95b889946
Signed-off-by: Sooyoung Ha <yoosah.ha@samsung.com>

Allowed 'sdb root on' command even if the UID of SDBD is non-root.

 - file IO(pull/push), shell service can be worked by root user with restricted priviliges.

Change-Id: I37f4248443bd7f5231ec99e0d1737cd556b749b3
Signed-off-by: shingil.kang <shingil.kang@samsung.com>
Signed-off-by: Sooyoung Ha <yoosah.ha@samsung.com>

Add capability 'sdbd_rootperm'

Change-Id: Ib63b4f3605506f62e3a8e8d7e32fb5937062ca3c
Signed-off-by: shingil.kang <shingil.kang@samsung.com>
Signed-off-by: Sooyoung Ha <yoosah.ha@samsung.com>

package: update version (3.0.15)

Change-Id: I0085cc5980ebf1679f2c0193953c380831cd9ac7
Signed-off-by: Sooyoung Ha <yoosah.ha@samsung.com>

sdb.h: remove duplicated member

Change-Id: I11f7086dd60cefbe36be2a46b9e92e2585b14cf1
Signed-off-by: Sooyoung Ha <yoosah.ha@samsung.com>

package: update version (3.0.14)

Change-Id: Ia07a267448ac269c28c9f216b21ba678066f7f04
Signed-off-by: Munkyu Im <munkyu.im@samsung.com>

Revert "Revert "remove mkdir again for sdbd log file""

This reverts commit 22ee2bf467e3e798ee9b0811fef172e6b231f93a.

Change-Id: Ibdb42d07d508fe53f49ba75f4d0c2a9bff7d9df9
Signed-off-by: Sooyoung Ha <yoosah.ha@samsung.com>

Revert "Revert "remove mkdir for sdbd log file""

This reverts commit bdaaa8e914baf89a727c013d6b3e00a81a100aff.

Change-Id: I980f16a44b33c0e8a60be956d04e9d36541d81ca
Signed-off-by: Sooyoung Ha <yoosah.ha@samsung.com>

Revert "Revert "remove smack_setlabel function usage for security reason""

This reverts commit 9acf96cbd8e41c699e2f059c1bef256910215178.

Change-Id: I75479ca94a011a79764556da3776587addf413c1
Signed-off-by: Sooyoung Ha <yoosah.ha@samsung.com>

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

This reverts commit b3894c7aa4e819c5cde88c7d389fb8300af6fdb1.

Change-Id: I98cbf82cdc47392e8c3b8038c0e395bcc9e6ac3b
Signed-off-by: Sooyoung Ha <yoosah.ha@samsung.com>

cap: Add new "pkgcmd_debugmode" capability for pkgcmd debug mode

To support screenshot,
App should be installed with "pkgcmd -G" command.
It is activated by enabling "pkgcmd_debugmode" capability.

Change-Id: I0c65fbff2155f6d96cc1cfc39102844fce7c4ea6
Signed-off-by: Munkyu Im <munkyu.im@samsung.com>

fix a bug that device is still offline when reboot (USB connected)

modify broadcast_transport (send device status) not to send packet to
offline devices

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

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>