Add Requires=local-fs.target dependency to tizenid.service

In emergency mode, local-fs.target always fails.
So, you have to check if local-fs.target is successful.

Change-Id: Ic353d30a7ed7c058d0e05f890c96274d539d48b8

Set the plugin directory factoring in architecture

- armv7l  : /usr/lib
- aarch64 : /usr/lib64

Change-Id: I07804cb9dd9e996710a5d11f84772d4bc2fa9432
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Prevent buffer overflow on reading value

Change-Id: I4a6d5abce72c4f2165a0d190068ef75157cf4c35
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Integrate files into N files to reduce total file size

- Although the size of value is small, each key file takes a file block
  - Block size is 4kB
  - 270 model-config keys takes 1.5MB
- To reduce file size, store many keys into the same file
  - To find file name, use hash function
  - In current, store 10 files

Change-Id: I7140e732d7dde1e6c6ad65b28f5f7a064502d9f5
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Check the type by special function for conditions

- In current, only tizenid use it

Change-Id: I3c985a7d848f40fcacbfd5c5846c7dd082478188
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Read value from per-key file

Change-Id: I0cb6e0607961fe7660b8234f5ae32f90e5294530
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

change version to 0.5.2

Change-Id: If7ee54ab008c66b7e195c97a665be5e9c0073e9b

add multi-plugin functionality

Change-Id: Iacef58acb8b2e9dae93a7f2bd63813c82a1ce992

bugfix: fix type mismatch for custom key

Change-Id: Iacef58acb8b2e9dae93a7f2bd63813c82a1ce993

Fix: Do not use source argument length in strncpy()

This fixes issue found by gcc 9.x, where strncpy's length
argument was based on source argument length.  This commit
fixes possible bufer overflow.

Change-Id: I933e999d4919e1dbce2834138a612c9f97f916a4

Explict type cast according to iniparser 4.1 API change

From iniparser 4.1, iniparser_getstring retrun "const char*" instead of "char *"
To support the new API, explict type cast was added.
Although system_info_db_init requires "const char*", original codes was kept to minimize modification.

Change-Id: I3c0ee077500160b38a80c0257ba1b77585a81187

Update guide URL

Change-Id: I82fe9e57f395e00e979d25b76e70a15c41653a01
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Rename gcov package

Change-Id: I596e69f5d457254ea1b58f6afa6ecbfeab25464a
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Make gcov package

- To make, use "gcov" build option

Change-Id: Ic6cf306d62d48f187c97c8c327059755ff4c2d31
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Create system-info-test

- Input key information, then prints value

Change-Id: I0dac252d8fb3d518508113978def523d4987cca1
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Write debug log when the key isn't stored in DB

- The keys supported by system-info plugin aren't stored in the DB
- In this case, system-info always write error log though
  it returns right value (with ERROR_NONE)
- To prevent misunderstanding, change the log level
  - In real failure case, you can check the "Invalid key" log

Change-Id: Ia78cb2e87c5ee06cf20f3718694d262c7df6da90
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

tizenid.service: Add RemainAfterExit=yes

tizenid.service should be run only once at boot time.

Change-Id: I9390f63fedf25624bab3ad3090ce2362cd6b7dbd

Add LCOV comment for not used function

- destroy_key_value is called when hashtable is removed
- However, there is no scenario that this hashtable is removed

Change-Id: I7356d46b9bf84e083e8cf371854b4b5a54afa2f4
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

release : 0.5.1

Change-Id: Iab7ad9b9bc5651330329c901b0a831b8bdc5bc83
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Remove deprecated notice file

- format_uuid_v3orv5 is deprecated since Tizen 3.0

Change-Id: I40489026c193543e32553fb23109bfa3975e7d77
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Make some macros consistent

- Related with model-config and DB

Change-Id: I12b8b9629d7993a7d11a55bfcbd21342c278d570
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Merge remote-tracking branch 'origin/tizen_4.0' into tizen

Change-Id: I9c6e9bfaca6cc3ac897e8210e8926abc8f9e3809
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Fix model_config directory discretionary access control

Basically, directory DAC should be haven execute option.
So model_config directory will have it.

Change-Id: Iab3ff32c19f4b4015aebbf1546cd2b99bb40cb67
Signed-off-by: Donghwan Jeong <dh.jeong@samsung.com>

init_db: add system_info_db parameter for make new db by another path

It will be supported that the content can be chagned in system info at run-time.
So, system_info_db can make new system-info-db file by another path.

system_info_init_db -i --input=PATH   Make System info db by the PATH
ex) system_info_init_db -i /etc/config/model-config.xml

The new db file will be saved to '/opt/system/model_conf/system_info_db' path

Change-Id: I2ddde7e313ff5cc547b3592109fe1cf29ddbdd7b
Signed-off-by: Donghwan Jeong <dh.jeong@samsung.com>

Write RUNTIME_TYPE log with debug level

- If app launcher doesn't set RUNTIME_TYPE, this log will be written
  whenever system-info API is called for getting boolean value

- For developer to give control, modify this log's level

Change-Id: I2d393efaf73d0c7243cc321753825a684c00231e
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Merge remote-tracking branch 'origin/tizen_4.0' into tizen

Change-Id: I75d6c2cf7a89051293838f20e440a0049790975d
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Fix memory issue about the value read by GDBM

- Limit duplicated size
- Free after using

Change-Id: I530683f386bb64d402c8277824051c91aa003330
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Fix memory issues

- Prevent accessing out of array
- Free before exit

Change-Id: Ifbe15353753438efb1b0f79951121d717cd57038
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Fix memory issue about the value read by GDBM

- Limit duplicated size
- Free after using

Change-Id: I530683f386bb64d402c8277824051c91aa003330
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

init_db : allocate one more byte to boolean value

- For null character, write (len + 1) bytes to the DB
- However, only len bytes are allocated for boolean value
- To prevent buffer over-read, give a spare byte

Change-Id: I5582e0ac5c1d5837ce9ac1b295762814011133f0
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Remove build option related with upgrade script

Change-Id: Ie1e01c0bb0ab7354c9dc32fee4f5a6a5e1b3a218
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

script : remove upgrade script

- No more support 2.4 -> 3.0 upgrade

Change-Id: Ifb2024a432fa5593f27d7d154f5ba4bcb5300cef
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

update_db : remove system_info_update_db

- It is for emulator to chroot before system_info_init_db
- But, with using busybox, we can use chroot in the initramfs
- To apply with emulator, the prerun patch should be applied first
  - https://review.tizen.org/gerrit/#/c/143420/

Change-Id: I49f12420005ef8860fbb65efb45c57179eadfc0d
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Merge "Use memcpy to prevent buffer over-reading by snprintf" into tizen

Use memcpy to prevent buffer over-reading by snprintf

- snprintf calculates return value by checking the length of format string
- Therefore, if format string is non-terminated, snprintf can over-read
- To prevent, use memcpy and write null character

Change-Id: I9ba837d8e22313be6e34ba39e4ccaf7743166d89
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Make version information for update at imaging

make_info_file.sh constructs platform info file during imaging.
Platform update needs formatted version information which also
could be constructed at imaging time.

Change-Id: Ibee71bef3a5811fc725d1648ea69b88be384a8bd
Signed-off-by: Sunmin Lee <sunm.lee@samsung.com>

Enhancement system-init-db for updating specific lang feature

Now, system init db tool can update bool value for specific runtime.

The feature will be updated according to following command.

Whole runtime feature update : -k KEY -t bool -v VALUE
C# runtime feature update : -k KEY -t bool -v VALUE -l csapi
C runtime feature update : -k KEY -t bool -v VALUE -l capi
Web runtime feature update : -k KEY -t bool -v VALUE -l webapi

Change-Id: Iad562a040778b4b1b76492ad6f06e6043edee234

init_db : update boolean to modified value

- The lang option of update is removed because of the lack of default value

Change-Id: I8568e369a4ddd140b070256c1a4d199090cd58aa
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

init_db : write boolean to 'T' or 'F' instead of 0x1 or 0x0

- To prevent misunderstanding false as the end of string
- In addition, we can get the readbility

Change-Id: If94ec9337f4360f696c242664e5d32401da5f934
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Version up

Change-Id: I45c65e0d5645a881dfa009eda96fbde83614129c
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Fix a bug of buffer-size using when checking tizen-id validation

It should be ID_LEN(28)+4 because 32 byte is maximum.

Change-Id: I0a877b9b01c7fa1184308598f0ce21c03f1707d9

Generate tizenID with libuuid instead of openssl

- To detach openssl

Change-Id: I55a6cbbd686857a94d0ff7cb9806759db4c40f03
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Support language runtime property for boolean feature

- Some keys are needed to have different value depend on runtime
- To support different value, model-config will have runtime property
  - e.g. < ... capi="on" webapi="off" csapi="on">VALUE</key>

- init_db stores the value for all runtimes with default value
- Before system-info reads key, it check RUNTIME_TYPE
  - If RUNTIME_TYPE is invalid or NULL, return default value

Change-Id: I247245270ddf58ae2da5254a23b0fa2188e77899
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Switch off UBSan for update_db

Change-Id: I6ffd4ce833bd72c1ee66cdc77f8139b63ce618a0
Signed-off-by: Denis Khalikov <d.khalikov@partner.samsung.com>

Switch off ASan for update_db

Sanitized binary cannot be built with '-static' since Address Sanitizer intercepts function calls from libraries.

Thus, system-info is built in both Address Sanitizer On environment and Address Sanitizer Off environment. Then, system-info-update-db built without sanitized build option is installed.

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

Correct broken link

Change-Id: I6fc657e9a22bd9d2e4f9aa35933de3b697536aeb
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Fix invalid type specifier

Change-Id: I5ef279c47e01468417c347faf19573dc27bee1ad
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

update_db : do chdir after chroot

- To prevent from accessing outside of chroot directory

Change-Id: Ia951443f9a0f4da12596640361288519b94a38c6
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

remove the dependency of model-config

   It will be handled by ks file.

Change-Id: I0f3bdbf34e0734725621dd614aa3ab439d6221d5

add Recommends for model-config

Change-Id: Iae9ab03093e7e248901bfb81111f59e5f3749223

revise packaging for unified build

Change-Id: Idcc56926fbbb07840d25da2117f49eb42d293517

Fix spec file and add manifest file for extension-emulator

Change-Id: I01f72716a252b840e81a7266c64ccfc18524f1ab
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Add LCOV comment for not supported API

Change-Id: Ieda1db7075020d61bd6cb7d11c57263ad318cc90
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

[Tizen] Remove emulator build dependencies

- This is for Tizen 4.0.
- Added backward-compatibility that does not deteriorate 4.0 Configurability

- When you SR this, you need to create JIRA-TRE issue of:
  : add capi-system-info-extension-emulator for all(common/mobile/wearable/ivi/tv) emulator

(It's add, not replace.)

Change-Id: I8fcbd4760846ba03d9ea2adfbbb6117ab73e714e
Signed-off-by: hk57.kim <hk57.kim@samsung.com>
[Fix build error]
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Modify uid/gid of tizenid

- system_share will be deprecated

Change-Id: I92e20ba86c81b08358fee1829c6ceed2f43c2ca0
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Merge branch 'tizen_3.0' into tizen

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

pc: add prefix for the pc file

Pc file uses PREFIX macro, but there was no place to define the macro.
Now, The macro is defined in the CMakeLists.txt file.

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

Merge branch 'tizen_3.0' into tizen

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

pc: add prefix information

prefix information is not the necessary information.
But the information is used in other place, and thus
the information is added.

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

Proofread header file

Change-Id: Ia4e206d3ea8d0364d09d3c37ed1d411b91c23bfc
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Merge remote-tracking branch 'origin/tizen_3.0' into tizen

Change-Id: Iaca788a3fa7b68277ed05242875982fec2f23618
Signed-off-by: pr.jung <pr.jung@samsung.com>

Merge "Fix for memory leak" into tizen_3.0

Fix for memory leak

- gdbm_fetch returns a d_data and the actual value is in d_data.dptr
- Memory allocated to d_data.dptr needs to be freed

Change-Id: I774f2b16bbfbeb1cc500976aa266143aa56d5dbb
Signed-off-by: pr.jung <pr.jung@samsung.com>

model-config: add dependency for the model-config.xml

During image creation, model-config.xml file needs
to be installed before system-info. Thus the dependency
is added.

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

model-config: add dependency for the model-config.xml

During image creation, model-config.xml file needs
to be installed before system-info. Thus the dependency
is added.

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

Upgrade: Do not use systemctl

The system offline update is going to be done without
basic.target. However, systemctl command requires basic.target
internally. So, replace it with a command.

Change-Id: I84fecf4ad727a82506cd4eab9ce091fa29b35d03
Signed-off-by: Sunmin Lee <sunm.lee@samsung.com>

Merge branch 'tizen_3.0' into tizen

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

db: create db in the post script

Previously, the db was created after installing all packages.
However, several packages want to use the system information
during installation. Thus the db is created once in the post
script, and db is updated after build information is created.

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

Merge branch 'tizen_3.0' into tizen

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

Upgrade: set RW update script number

+ revert patch using binary instead of systemctl

Change-Id: I728645cb1a8d15b96dadf68942935b4925e5d454

upgrade: remove systemctl command from upgrade script

- During upgrade, systemctl cannot be used. Thus tizen_id
  needs to be called directly.

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

Add lock for multi-thread cases

Change-Id: I7a362ca4341b9e0c5fdc765980b885050aae9f8c
Signed-off-by: pr.jung <pr.jung@samsung.com>

upgrade: fix upgrade script directory

Change-Id: I4154cdea4cc49faaff19f124bcb488ba408dc5c4
Signed-off-by: Sunmin Lee <sunm.lee@samsung.com>

upgrade: add upgrade script for tizen id

- tizen id is created on the system partition,
  and thus the permission of the tizen id path
  needs to be granted to make tizen id properly.

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

add install-time dependency for security-config

because security-config provides system-share

Change-Id: I7fd870a52134088dd865a7f278a528a5ad36d240

Merge "Tizenid service SHOULD use std io because it is executed at sysinit.target" into tizen

Tizenid service SHOULD use std io because it is executed at sysinit.target

Change-Id: Ie7da8ed400c234bf0625265f7bd498b5aaac1836

Add hash table

- Find hash table before DB

Change-Id: Icad177d108e3b990d33e81df8f8dce3cdda4c26d
Signed-off-by: pr.jung <pr.jung@samsung.com>

tizen id: change user and group of systemd service file to system_share

- Tizen ID is made with the system_share user and group.

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

db: add db update command for Emulator

- In the emulator environment, the display resolution
  can be changed before the first boot. Thus the command
  system_info_update_db is used to update the system info
  values before the first boot.

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

Merge "Fix pkgconfig file Version: field" into tizen

Fix pkgconfig file Version: field

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

remove useless definition for prefix for cmake

 - incompatible with yocto build native

Change-Id: I07bfa938cf3dfb8335f2f2693337386a28b52942
Signed-off-by: Ronan Le Martret <ronan@fridu.net>

remove hard coded path to pkg-config pc file

 - we need to generate the pc file with cmake macro for yocto native build.

Change-Id: I807a5c8e8d04cd5af22c9c188a3067291426239a
Signed-off-by: Ronan Le Martret <ronan@fridu.net>

plugin: remove sample plugin

- Sample plugin codes existed just for guide.
  But it is unnecessary codes, and thus they are removed

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

tizen id: change tizen id path to set smack label

- Tizen ID is stored in the path which has transmute
  property to give smack label automatically

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

coverage: add comments to exclude from the coverage test

- Error exception routines can be excluded since they
  could not be occurred.
- Plugin handling routines can be excluded since Platform
  does not have the plugin.

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

tizenid: change rand() to rand_r( to fix the thread unsafe issue

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

common: set PATH env

- PATH env need to be added since system PATH env
  can have weird paths.

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

tizen id: use rand() instead of /dev/random

- /dev/random is powered from actual entropy
  gathered by the environment. If the internal
  entropy pool is exhausted, it is need to wait
  for a while to get random value from /dev/random.

- Not to wait to read random value, rand() is used.

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

common: apply Tizen coding rule

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

tizenid: move tizenid.service to sysinit.target

- Previously, tizenid.service was started at multi-user.target.
  Thus several apps tried to access tizen id before making it,
  and failed to access tizen id.
- Now, tizenid.service is started at sysinit.target. Thus
  tizenid is made just after /opt is mounted.

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

gdbm: reduce cache size to 10

- Before reducing cache size, HEAP size allocated
  was 428kB with test app.
- After reducing cache size, HEAP size allocated
  is decreased to 64kB.
- The performance to read data is not decreased.

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

db: change backend of system info from xml to gdbm

- Previously, Each app using system-info opened and parsed
  the model-config.xml file. However the size of libxml2 is big
  and xml parsing is required whenever each app gets system
  information.

- Now, gdbm is used to store the information of model-config.xml.
  Thus memory size is decreased and read db is faster than
  parsing the xml file.

  * App with libxml2
    - PSS: 231 KB
    - Average time to get a value: 14,331 usec

  * App with gdbm
    - PSS: 176 KB
    - Average time to get a value: 408 usec

- The db using gdbm is created at image creation time.

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

tizen id: change path of Tizen ID to RW path

- Previous: /etc/tizenid (RO)
- Now: /opt/etc/tizenid (RW)
- Previous path is RO and thus Tizen ID cannot be made
  at runtime. Thus the path is changed to RW path.

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

Merge "build info: change ID and Release" into tizen

Merge "tizen id: reset tizen id if there is no content in the id file" into tizen

Merge "tizen id: reset Tizen ID if it is not a normal file" into tizen

build info: change ID and Release

- ID format is changed to get the value from Version.Build
  = Previous Build ID example
    : tizen-mobile_20160308.8_mobile-wayland-armv7l-tm1
  = Changed Build ID example
    : tizen-mobile

- Release value is changed to get the value from Version.Build
  = Previously, the value was created by removing alphabet
    from Version.Build. However the Version.Build could contain
    other numbers between alphabets. Thus Release value
    could be weird.
  = Now, Release value is parsed from Version.Build by
    the Build.Date value.

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

common: apply Tizen Coding Rule

- Space before the square bracket is removed

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