libdlog : fix invalid log_fds access

- If all log device paths are given and failed to get main fd,
 libdlog will try to access log_fds[LOG_ID_MAX]
- No more access log_fds[LOG_ID_MAX]

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

Free config data and close fd before terminated

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

logger : fix '-s' option is not applied

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

Revert "Fix passive logging issues"

This reverts commit e5c03b719438501f9ccca890092b961e73ac164e.

New patch with fixing bug will be uploaded

Change-Id: I1d7e18dda7bcd3bc6bf9b19c1d30a0b3579a3858

Fix passive logging issues

 * the -s parameter now works
 * filtering actually works

Change-Id: I9c6552b8b8fb7cbaca6afe3bcccf0553b81eb1fe
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Fix epoll_ctl erroring out

Do not call epoll_ctl on FDs representing actual files,
which causes the call to error out with EPERM.

This use of this mechanism was twofold; first, pipes
need to actually be added to epoll (to see if they're
full). The second use was to discover whether a descriptor
represents a file. This information is known a priori,
which removes the need to make calls doomed to fail.

Change-Id: Ic11c211ee88e70899998e794847b79afc0a90d9a
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Proofread header file

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

Check dump file size during initialization

- Until now, dump file is rotated after writing rotate_size_kbytes
- If writing less than rotate_size_kbytes and rebooting are repeated,
 dump file size can be infinitely increased
- To prevent this, large dump file should be rotated before logging

- Meanwhile, size conversion macro is defined

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

logger : prevent out of range

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

logger : fix about filtering

- TAG:PRIO filter will be applied

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

logger : fix invalid checking

- Return value is changed at the 3e8ab4998beddd0cf61486bb2c3f06ff9f646579
- Therefore, condition statement should be changed

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

dlogutil : do not sort when it prints logs about only one buffer or tag

- Each dlog buffer is FIFO queue, so dlogutil don't have to sort in this case
- Also, we can assume that logs having same tag are stored in the same buffer

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

Fix minor issues.

- Remove unreachable code
- Add error check code
- No more ignore conf value about buffer size

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

Upgrade: adjust upgrade script number

Change-Id: I425c22266e8b3c5e030600c49a5f10f7e03837b7

Merge "logger: Simplify binary log file header writing" into tizen_3.0

Merge changes I274d815f,I4c831e85,I997c4d72,I02ebfd88 into tizen_3.0

* changes:
  logger: Avoid returning EBADF in normal operation
  More useful return values for shared functions
  Catch allocation failures
  logger: refactor writer servicing

Merge "logger: fix closing fd 0" into tizen_3.0

Merge "logger: fix socket owner change" into tizen_3.0

logger: Simplify binary log file header writing

Additionally, add return code as writing may fail
and handle the return code in invoking functions.

Change-Id: I7c1434fe87b6f6a50ea986307e935efd87775796

logger: Avoid returning EBADF in normal operation

EBADF has been used only to signal that file should be removed from
the list - this commit makes this explicit.

Change-Id: I274d815fd95ab43aa13f14d36286d17672e42e83

More useful return values for shared functions

Now returning 0/-errno instead of 1/0

Change-Id: I4c831e85c8ec7e93f183f6990839afdb63ce0a63
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

logger: fix closing fd 0

An unused fd whose value was always 0 was wrongly closed.

Change-Id: I8a70c0a447abd83f091993113354095ffd6446c1
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

logger: fix socket owner change

Change-Id: I742f332bcec3d01fefd2cae3f12d4f7f85878422
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Catch allocation failures

Change-Id: I997c4d72f3ba413246085b60e49ffeaac1753d7b
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

logger: refactor writer servicing

 * inflict consistent return values - all writer-related functions now
   return 0 on success and negative errno code on failure.

Change-Id: I02ebfd88fd044ee083675ba9ed88c403530ca786
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Upgrade: adjust script number

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

Revert "Upgrade: adjust script number"

This reverts commit 25acf3aa801bcb858517aea3194cf7aa9c547adf.

Change-Id: Id799dab1c07023e8298ec45037851f2c07a992e0

Upgrade: adjust script number

dlog upgrade should be done before 300.pkgmgr.patch.sh

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

dlogutil : add "-u" option for setting sorting buffer size

- dlogutil -u SIZE
- If "-u" option isn't used, set to the value of conf file

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

dlogutil : fix memory leak

- When dlogutil filters specific logs, not printed logs haven't be freed

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

libdlog : set initalize value to write function pointer

- If log_config_read() is failed (e.g. If '/opt' is unmounted),
  dlog_print function refer to null
- Therefore, it need to set to fallback function

Change-Id: I1002c795a12de062121f656a32414a739b994692
Signed-off-by: Jiwon Kim <jiwon177.kim@samsung.com>
[Remove comment violating Tizen coding rule]
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Fix vulnerability: NULL check

Null checking for the returned value

Change-Id: Ia459629462b9f0e6cf8501f67f973a92c60b3caa

Fix vulnerability: log_config_set()

This patch forces null termination of e->value string.

Change-Id: I32fba2f23ef2a1b4c725585e2835d2a5530f2d27

Merge branch 'tizen' into tizen_3.0

Change-Id: Iaabf10b234499155e98ab354b4c2084afc3e93fd

libdlog : control platform debug log by modifying conf file

- If debugmode is set to 0, libdlog doesn't write platform debug log
- Debugmode doesn't impact on Public API (dlog_(v)print)
- Normal user can't modify it

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

dlogutil : print logs when sorting buffer is full

- When specific app prints huge logs without a break, dlogutil also wait endlessly
- Therefore, we have to empty buffer when it is full

- Meanwhile, if you feel that time to fill buffer is too long, you may reduce buffer size
- However, there is a trade-off about sorting buffer size
  - Having larger buffer guarantees sorting quality
  - Having smaller buffer prints logs fast
- Therefore, we may give user a choice

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

Merge branch 'tizen' into tizen_3.0

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

Release up

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

Tests: add a unit test for pipe message creation.

Change-Id: Ic95a7abea1d9933a42605c8f192d2093deb8b80c
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Pipe: move validity check to the dedicated area

  Ideally, this series of checks (null message, total length, priority bounds)
  would be a separate function shared by all back-ends. This will be done in a
  further commit.

Change-Id: I853c64a32e3ba5e6a72b28b5075f4fca39d6a15f
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Check our privilege.

Make sure the privilege reset succeeds.

Change-Id: Ic88703c2ac8d69eb1a173526c60363ba319820c6
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Merge "Set proper executable bits" into tizen

Set proper executable bits

 * add to scripts/dlog_upgrade.sh

 * remove from plaintext files; this includes source code,
   configuration, header, image, manifest, service, and other
   miscellaneous (eg. license) files (specifically, all files
   touched by this commit except scripts/dlog_upgrade.sh)

Change-Id: Icc1496147c5e78347dff8af5eade3c6cfe13bffd
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

dlogutil : print logs stored for long time

- If any logs don't be written, some logs stored most currently can't be printed
- When remainder logs haven't printed for long time,
 it will be printed without written time

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

dlogutil : modify timeout when dlogutil decides to terminate

- When "-d" or "-t" option is applied, dlogutil is terminated
 after printing remainder in the buffer.

- However, if the time interval with current log and last log
 is less than timeout, current log can't be printed.

- Therefore, timeout should be zero after decision.

- In addition, to prevent the situation that dlogutil can't decide
 because of continuous logs, waiting timeout also be decreased.

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

Disable to log to console & stdout for syslog

Change-Id: I7ff528856ee35e4f8883edbd18f592038dfc9085

dlogutil : check elapsed time by comparing both ends

- Pipe backend uses clock_gettime and Logger backend uses current_kernel_time
- Because dlogutil uses clock_gettime, it can't check elapsed time at the Logger backend
- Therefore, for portability, dlogutil doesn't record current time

- To prevent underflow, use long variables

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

Merge changes Ib648c1bf,Idb9716c8 into tizen

* changes:
  dlog_logger: Drop redundant User/Group settings
  dloginit: Execute dloginit service with all filesystems mounted

Merge "Micro-optimisation" into tizen

Merge "Fix device file path" into tizen

Fix a deadlock

Change-Id: I606ebf58c071be4974be58933799d1aadc214342
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Fix device file path

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

dlog_logger: Drop redundant User/Group settings

User and Group default to root for system services.

Additionally, reformat note about TZ variable setting.

Change-Id: Ib648c1bf47169c3da5476e4612e11b5d8826ded3

dloginit: Execute dloginit service with all filesystems mounted

dloginit needs access to /opt/etc/dlog.conf for it to read
administrator-provided config correctly.

Change-Id: Idb9716c80786a721b54bf5f67a7cc38065feeb59

Micro-optimisation

Change-Id: I790920282f4d6b4857d3a5b0535e5a9c202c8a70
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Report meddling with libdlog internal state

Change-Id: Ife49638d2f0f7fdca69b015d51ed8c30e3a32834
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Extract message creation to a separate function

Change-Id: I03a99b40599164f7eb595e0824e02551c17e7462
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Merge "Fix code issues found by cppcheck." into tizen

Merge "Add unit tests." into tizen

Merge topic 'dlog-pipe-fix' into tizen

* changes:
  pipe: Only take up file descriptors when needed
  pipe: Fix multithreaded pipe file descriptor reacquisition
  Ensure dlog is started before regular Tizen services wanting to use it
  Prevent segmentation faults and memory leaks
  Fix libdlog initialization with multiple threads

Fix code issues found by cppcheck.

Change-Id: I79547320b0bb566efba904163094861d7514e83d
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Add unit tests.

Change-Id: Ic1c02937f6cf6577ce63aa5961f76a5b54f0af99
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

pipe: Only take up file descriptors when needed

Change-Id: I7ade660e8e2273654ed8b782f42870824b33256e
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

pipe: Fix multithreaded pipe file descriptor reacquisition

Change-Id: I6d64a9f9adea2334dcaa67cbe2e6c6d98cb58b45
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Ensure dlog is started before regular Tizen services wanting to use it

Change-Id: I9b80c2ae97b428be3309fbc30eb000d7f8716fa9

Prevent segmentation faults and memory leaks

Change-Id: I51125aa059f028280f21cde2e72053f81ecfdbb6
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Update release number

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

Merge "dlogutil : fix that some logs overtake previous logs having same time" into tizen

dlogutil : fix that some logs overtake previous logs having same time

- Two logs can be written at the same time (having same sec and nsec)
- In this case, the log written afterward should be located backwardly

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

Merge "pipe: Also reconnect when last write was for bad fd" into tizen

Fix libdlog initialization with multiple threads

The backup null log handler is now set only on initialisation failure
instead of always being set at the beginning and overridden later.

The initialisation check does not care which particular handler is installed,
so threads which attempted to write a log while another thread was carrying out
initialisation could be directed to the null handler.

Change-Id: I3aaf6dcad8f72611e413969d48f0f68d44a7c308

pipe: Also reconnect when last write was for bad fd

Change-Id: Iaf7f2fe44383a0395fd7199564aa3a28cf44298d

Version up

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

Merge "dlogutil : sort logs regardless of backend" into tizen

dlogutil : sort logs regardless of backend

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

Add path variable in the script file.

- For security issue

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

Merge "pipe: Fix recvmsg() error handling" into tizen

Merge "pipe: Ensure pipe fd is opened in blocking mode" into tizen

Merge "pipe: Fix fd leaks during connect" into tizen

pipe: Fix recvmsg() error handling

This fixes problem with recvmsg() error return code 0 being returned
as proper fd id (0, stdin).

Change-Id: Ia9be0b0b5d12ab202ea52c4153b749e1305a83c4

pipe: Ensure pipe fd is opened in blocking mode

We need to either use blocking fd for write pipes or
handle EAGAIN errors explicitly.  First solution is
more effective.

(NOTE: Reader sockets should use non-blocking fds
 as readers - dlogutil - to not stall the daemon.)

Change-Id: I5a1043676bb963c7ecb722914d1933839d857093

pipe: Fix fd leaks during connect

Change-Id: Idfb4c68852456a6011194c7d0c9d23874bf202bf

Use right variable

- Use unsigned char for storing only positive
- Only compare with not null string

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

Revert "Modify variable type"

This reverts commit e793a48a6386c590dac6c541413603fa97835f45.

Change-Id: Ie2ae3e54a8aabbb0392c634678486fe23f571621

Modify variable type

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

Revert removed declaration

- In some environment, dlog-internal.h is excluded
- Therefore, dlog.h have not to rely on the declaration of dlog-internal.h

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

Add dependency to each sub-packages

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

Version up

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

Merge "Fix a typo" into tizen

Merge "KMSG handling more robust." into tizen

Cleanup unused code

Change-Id: Ie48b77b751be1f06b297ccecf89e73287a1e1488
Signed-off-by: Michal Bloch <m.bloch@samsung.com>
[Rebase]
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

library integration : build all backends at one time

- Backend is selected by installing dlogbackend-BACKEND
- If any dlogbackend-BACKEND isn't installed, pipe backend is selected

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

library integration : remove backend dependent macro

- To use dlog with all backends by building only one time, backend dependent macro should be removed
- Instead, we will find backend information in the conf file

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

Fix minor issues

- Use thread-safe function
- Guarantee inserting null character
- Don't write exceed value to char variable

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

Fix a typo

Prevents grep attempt ruination

Change-Id: I3e036d6d0820a8be56e2f07a804c50e7c18138ea
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Logger : set right permission to socket and log files

Change-Id: Ia5793ab0ce1e15da0d16c0685258f965a64e87a2
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>
[Split logger configuration & file parsing]
Signed-off-by: Karol Lewandowski <k.lewandowsk@samsung.com>
[Bug fix]
Signed-off-by: Kichan Kwon <k_c.kwon@samsung.com>

Set debugmode by checking debugmode file

- If debugmode is enabled, fatal log can be written
- Else, process writing fatal log is aborted

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

KMSG handling more robust.

 * the parser handles malformed logs
 * util no longer discards parser result

Change-Id: I2e706b970b88717265f552703b0774a2e276519a
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Fix KMSG EPIPE handling

KMSG devices return EPIPE when the cyclic buffer rotates.

Change-Id: Idb27bccaee8bb14959e52fefaa70cf076c137bf8
Signed-off-by: Michal Bloch <m.bloch@samsung.com>

Merge "Update dlogutil help" into tizen