Merge "Adds error handling when a change of user privilege has failed." into tizen

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>

Fixed a bug where sdbd shows incorrect error message

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

Fixed build warnings.

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

Fixed sdk_launch script
- Supported multi user feature

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

Add permission of the log group to sdbd service.

- To obtain permission to run the dlogutil, the log group
  permissions has been granted to sdbd service.

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

MISC: package version up(3.0.7)

- Add the check routine for multi-user support.
- Add the sdk_toolpath member to sdbd capability.

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

Add the check routine for multi-user support.

- The multi-user support information is obtained
  from tizen-platform-config.

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

Add the sdk_toolpath member to sdbd capability.

- The sdk_toolpath member includes a path that
  tool of the SDK is saved.

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

Removed libprivilege-control dependency

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

Removed unnecessary codes

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

Fixed sdk_launch script (merged from tizen_2.4)

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

Merge "Removed hard-coded path to support multi-user." into tizen

Removed hard-coded path to support multi-user.

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

Modify the fixed SDK constant(developer).

- changed to obtained using the platform API(tizen-platform-config).

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

Merge "Fixed sources reported by static code analysis tool." into tizen

Fixed sources reported by static code analysis tool.

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

MISC: package version up(3.0.5)

- Fixed the incorrect version information for profile_command.

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

Fixed the incorrect version information for profile_command.

- Modified to 3.0 from the 4.0.

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

Fixed build warning "implicit declaration of function"

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

Merge "Fixed an incorrect function call for usb function." into tizen

Fixed segment fault error on arch64

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

Fixed an incorrect function call for usb function.

- Whether usb protocol is supported or not in sdbd, initialize
  the function pointers for usb.

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

sdbd: remove sys-assert dependency

- sys-assert is not used in Tizen 3.0.
  Thus the dependency is removed.

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

Fixed build error

- Fixed %changelog not in descending chronological order

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

Fixed build error

- add changelog section in RPM spec

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

Remove changelog

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

Add change log

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

Fixes for issues reported by Static Code Analysis

- Variable atomicity
- Bad sizeof usage

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

register vconf callback to be notified status of power lock

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

Merge branch 'tizen_2.4_merge' into tizen

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

prevent buffer overflow when reading struct array

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

Fixed codes which may occur buffer overflow or resource leak

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

Fixed codes which  may occur buffer overflow or resource leak

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

Remove unnecessary code for smack.

- change to the use of libsmack library.

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

Applied ASLR

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

sdb: do not init usb protocol is the profile is TV

- In the TV target, there is no usb connection for sdb.
  Thus usb protocol init operation should be removed

Change-Id: I4d2d6cd631d39468ab283eb47d8502ad04f7aa45
Signed-off-by: taeyoung <ty317.kim@samsung.com>

service: remove dependency with default target

- Default target has dependency with graphical target.
- If a service file whose type is oneshot in basic
  or multi-user target is blocked by any reason,
  launching sdbd is also blocked. Thus developers
  cannot develop their modules or apps.
- Now, the dependency is removed and thus sdb can be
  enabled without any dependencies

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

Change SmackExecLabel to SmackProcessLabel

Since systemd v219 Tizen-specific SmackExecLabel option was dropped in
favour of SmackProcessLabel option from upstream.

Change-Id: I36946fc5d57d55efe23691cac67f82aa800cd2e2
Signed-off-by: Maciej Wereski <m.wereski@partner.samsung.com>

sdbd: Add tizen environment to sdbd process

When systemd launch the sdbd process,
it does not have any tizen environment value.
The tizen-system-env.service makes the /run/tizen-system-env from /etc/profile.d/.

Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>
Change-Id: I58dd8c37de649f217c1cdfd21afc0d456711db46

Merge "Support both functionfs and sdb gadget" into tizen

modify system-info key string

Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>
Change-Id: Ib9f5b04ba67326dc176aaa1183504ee840bd8dc3

packaging: fix invocation of usermod in post-install

Change-Id: I6e6716e157356319be8ec40b77701de10eea17d0
Signed-off-by: Stephane Desneux <stephane.desneux@open.eurogiciel.org>

Creation of the user account for the SDK

This replace the previously hackish creation of the
user handled by SDB for the SDK.

The current implementation creates the user on need at
installation time of SDB. If the user still exists, it
is not recreated to avoid erasing of previously existing
data.

The group developer is also created.
TODO: check that the group developer is needed.

Change-Id: Ie2a1fae481ad7f89f8349aafa1d19f858b8a1313
Signed-off-by: José Bollo <jose.bollo@open.eurogiciel.org>

Merge "Fix sdbd error: proc/1011/oom_adj is deprecated, please use /proc/1011/oom_score_adj instead" into tizen

Use %{_unitdir} macro to package systemd unit files

Bug-Tizen: TC-2069

Change-Id: I24445b2e71d9473f2afcdeb4813e4577060e9795
Signed-off-by: Geoffroy Van Cutsem <geoffroy.vancutsem@intel.com>

Align with Multi user feature, SDB tool should concider develloper as regular user

Develloper user should be owned by users group

Add Home directory creation
Linked with : https://review.tizen.org/gerrit/#/c/28178/
BUG=TC-320

Change-Id: I79fad0baac5c0bd763aa3d4d56a07d3f49be55e0
Signed-off-by: Baptiste DURAND <baptiste.durand@open.eurogiciel.org>

Support both functionfs and sdb gadget

Automatically detect the kernel interface to use for communication
with host.

Change-Id: I51834991063d9bbb686598490ddc7a0b607fc282
Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>

Fix sdbd error: proc/1011/oom_adj is deprecated, please use /proc/1011/oom_score_adj instead

Change-Id: Idb48c5f459ec995d42cc9d291d64473c416c1236
Signed-off-by: Mykola Tryshnivskyy <mykola.tryshnivskyy@globallogic.com>

Change sdbd running with User Smack label.

According the Smack rules in Tizen 3, sdbd should be
running with User Smack label. This will fix the issue
that Xwalk cannot access the applicaiton package pushed
by SDB.

Change-Id: I25624ef73413825d8ffd2c5d36e5aff3385a9f37
Signed-off-by: Alice Liu <alice.liu@intel.com>