log_dump: selective mode for dbus activation

log_dump running root in debug mode, and none_root with normal mode.
Simplify org.tizen.system.crash.service conditional compile by using m4

Change-Id: I04b734f722266264434b27e9fe7b17cdc73263f1
Signed-off-by: minsoo kim <ms203.kim@samsung.com>
Signed-off-by: Sunmin Lee <sunm.lee@samsung.com>

Adjust buffer size for address

The addr gets string of address consisting of begin/end
address and '-', null character.
So the length of addrees buffer should be set to:
 ADDR_LEN * 2 + 2 (at least)

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

Fix sscanf vulnerability

Specify a limit on the input string length
and keep the last byte for null character.

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

crash-stack: Fix Tizen Coding Style violations

Rules at: https://wiki.tizen.org/wiki/Native_Platform_Coding_Idiom_and_Style_Guide

This commit brings no functional changes.

Change-Id: Ic95fd2d1e5508d602fa24ea33933448c52ce625d

crash-stack: Change "ThreadID" heuristic to work on aarch64

Commit 82e3caa86 ("crash-stack: Find crashed tid by wchan")
introduced heuristic method of finding correct thread id, if
it was not passed by the kernel as parameter to crash-manager.

This commit modifies this heuristic to also consider "pipe_wait"
function in kernel to signify crashed thread.  This is what was
observed to be true on aarch64.

Change-Id: Iefa6d028f459ff9491f8e184433f913f39a096eb

crash-worker: Generate core only if /opt/etc/.debugmode file is present

Additionally, this patch moves actual path definition to one place.

Change-Id: I984ac2c67d8a22ed749f2b1945a1cef237b88ace

Merge "Removing dependency of tizen-debug-on/off service from sys-assert" into tizen

Removing dependency of tizen-debug-on/off service from sys-assert

Tizen-debug-on/off service should control enabling/disabling coredump generation.
When Tizen-debug-off service is executed, only callstack should be generated.

Thus, although disabling sys-assert, the services should be remained.

TODO : For disabling sys-assert, the services should be changed properly.

Change-Id: I617356cd31b51dbe4f103cb3118fefe454e717db

Merge "spec: change group(system -> root) on tizen-debug-off.[service|socket]" into tizen

spec: change group(system -> root) on tizen-debug-off.[service|socket]

Signed-off-by: INSUN PYO <insun.pyo@samsung.com>
Change-Id: I32605d431b3ff16cce61ae2dde7be262a1eac56f

crash-pipe: Drop all reporting functionality

Reporting is handled by crash-stack.

Change-Id: I6066c5a414fd2e98efbcc0ddc57f2f967cbfb859

crash-pipe: Introduce option to copy core using splice(2)

Linux splice(2) avoids user-space copy and is effectively faster than
regular read(2) + write(2).

According to (ftrace) test below splice is about 20% faster.

  sysctl -w kernel.core_pattern='|/usr/libexec/core-pipe --save-core /opt/usr/share/temp/core.%p'
  cd /sys/fs/kernel/tracing/
  echo do_coredump > set_ftrace_filter
  echo function_graph > current_tracer
  cat trace_pipe

/* splice */

 2) $ 2183078 us |  } /* do_coredump */
 4) $ 2127946 us |  } /* do_coredump */
 1) $ 2377004 us |  } /* do_coredump */
 1) $ 2088114 us |  } /* do_coredump */

/* read/write */
 1) $ 3088297 us |  } /* do_coredump */
 1) $ 2953330 us |  } /* do_coredump */
 1) $ 2647784 us |  } /* do_coredump */
 3) $ 2271315 us |  }

Change-Id: Id2c681a364ce96d98c70329abdf4a64f6d0b405d

Merge branch 'tizen_3.0'

This commit brings followings fixes from tizen_3.0 branch:
- Fallback to ptrace call-stack unwinder if sys-assert failed
- libunwind: Fix off by one in _create_crash_stack
- sys-assert: Fix handle leak of mmap

Change-Id: I422c4ddff0fb4dfb72738d70a7170080304081dd

crash-stack: Eliminate now-deprecated readdir_r

This commit fixes build break with new toolchain.

Glibc 2.24 deprecated readdir_r() - readdir() now recommended even
for multithreaded programs.  This commit fixes following error:

/home/abuild/rpmbuild/BUILD/crash-worker-1.0.0/src/crash-stack/crash-stack.c: In function 'find_crash_tid':
/home/abuild/rpmbuild/BUILD/crash-worker-1.0.0/src/crash-stack/crash-stack.c:891:4: error: 'readdir_r' is deprecated [-Werror=deprecated-declarations]
while (readdir_r(dir, &entry, &dentry) == 0 && dentry) {
^~~~~

Change-Id: I99f22a0de87f2539988e1669ae2149dcac74a4df

sys-assert: Fix handle leak of mmap

Unmap the memory after using it if it was mmaped.

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

Revert "Separate logdump from crash-worker"

This reverts commit c7826eefdccc5776eb0665cb0feb5d060e69fe96.

Change-Id: I398cf722d445b037d32d8116cc9c18a3a326b673

Fallback to ptrace call-stack unwinder if sys-assert failed

This commit enables ptrace-based call stack unwinder
(TIZEN_FEATURE_PTRACE_CALLSTACK).

When both SYS_ASSERT and PTRACE flavors are enabled
crash-manager will default to SYS_ASSERT, and fallback
to PTRACE only if former failed to generate the report
(eg. client application overwriten libsysassert's
signal handlers).

Change-Id: Ifc3626b7b46f47c3fd68efcb18d3900908ce3884

libunwind: Fix off by one in _create_crash_stack

This fixes call stack missing last element (usually _start,
in the case of invalid PC - the PC itself).

Change-Id: Ib1dae12bd06f1a4dfa1339fa6ea704bff05926c9

Separate logdump from crash-worker

In Tizen 4.0, the log_dump tool is supposed to be managed
in separate package.

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

Merge branch 'tizen_3.0' into tizen

Change-Id: I56715923d0ac0257736598e711d2c113548372cb

Fix vulnerability

- Enforce read size of read() less than buffer
- Do null check about opened bufferfile

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

Execute dump_systemstate in parallel

Since rearrange the order of crash modules, coredump had made
too late. It has caused several issues such as missing coredump
when something happened during dump_systemstate.
Therefore, execute dump_systemstate in parallel.

In addition, to avoid confusing launch crash-popup right after
whole crash works are finished.

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

crash-stack: Get executable name from cmdline

The executable name in crash-stack is different with sys-assert.
In case of some applications, "exe" has indicated real path of app
that makes it hard to know what the actual app name is.
It's better to refer to "cmdline" instead.

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

Print pre-ptrace info to buffer file

Some information had lost after ptrace.
However, the order of info also should be preserved.

To resolve this, that information are going to be printed
to temporal space first and will be merged in result file.

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

crash-stack: Find crashed tid by wchan

If tid is not offered by kernel, crash-stack would
find crashed tid by checking wchan value of each thread.

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

crash-manager: Utilize cmdline instead of core_pattern

There are limitations on using core_pattern variables to get command info.
 %e: *its length is limited (up to 16 byte)
 %E: in case of some applications, it returns not actual app binary path
     (it returns launchpad-loader)
Because of these reasons, use cmdline of /proc instead to get command info.
(It also makes a consensus with sys-assert)

* Refer to: http://lxr.free-electrons.com/source/include/linux/sched.h#L1663

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

Set suid_dumpable to debug (1)

As a platform policy, Tizen offers core dumping for
all process by default. However, until now some process
has got cleared dump flag by the system.
We modify it to these processes also could be dumpable.

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

crash-stack: Make output match sys-assert report

Change-Id: I7ecc9b045cc9b83d328d1d97bb08d4174c9d72ba

crash-stack: remove the core dump handling remains

Hopefully, there is no more rotting flesh.

Change-Id: I59003e8a38dd2817f78d23319848273615a0a5be

crash-stack: add file offset to output

Change-Id: Ib9f77b61d98a60e192877fad965a30ad8730af2f

crash-stack: disable unw_get_proc_name prediction

libunwind unw_get_proc_name function may return the name of
a label or a preceeding, nearby procedure; although the value
of the returned name plus the returned offset will
be equal to the instruction-pointer of the stack frame,
developers can find a exact symbol with related code position
in debug packages

Change-Id: I2471c2a9483b577844110f25128a6d539d39e5ad

crash-stack: integration test added

Change-Id: Iaac051671b89018dda37c212bb1c9da756879ed4

(Modified) Deal with Toolchain upgrade

(Revert "Deal with Toolchain upgrade")
This patch substitutes deprecated function "readdir_r" with "readdir"
rather than ignoring warning.

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

Deal with Toolchain upgrade

According to Tizen toolchain will be upgraded in 4.0,
several build issues should be resolved.
This patch deals with "Language rules stricting".

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

Merge branch 'tizen_3.0' into tizen

Change-Id: I62eb5bc520f06ea0daa13a7de4c3364d0a12695d

Change internal modules work

- crash-pipe: create core file only
- crash-stack: create .info file (call-stack)

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

Fix vulnerability

Check range of signo before use it

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

crash-stack: libunwind used on aarch64

Make use of "ptrace: Add support for GETREGSET" patch to libunwind
https://lists.nongnu.org/archive/html/libunwind-devel/2016-07/msg00001.html

Depends-On: I2e720e6697fbc1facf1d7547b398f5665b17731e
Change-Id: I0ce3886d271c9b78d2a44b99a628ccc502e6e400

crash-stack: Simplify signal info

It is hard to get signal info which caused dump using ptrace.
Therefore, crash-stack prints signal number only
which came from core_pattern.

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

Remove PD License

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

Adjust smack label

In response to security issue, restrict smack label of crash-manager.
In addition, adjust crash directories label to accord with
work of crash-worker.

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

Merge "log_dump: Restrict dbus permission" into tizen_3.0

Separate package dependencies of crash-stack

According to experimental variable, exclude unused package
dependencies from crash-worker.spec.

+ Bug fix:
   "crash-stack: Make ptrace callstack feature selective"

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

log_dump: Restrict dbus permission

log_dump can be activated using dbus method call.
Because this feature should be used by administrator only,
restrict the dbus activation permission.

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

crash-stack: Make ptrace callstack feature selective

In Tizen 3.0, the crash-stack using ptrace to generate callstack
will be offered as a choice.
Developers can enable this feature through setting a variable as on:
TIZEN_FEATURE_PTRACE_CALLSTACK

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

Remove SIGSEGV handling

SIGSEGV (signal number - 11) may occur double crash when sys-assert tries to unwind backtrace.
like that

If the double crash is generated, it causes useless backtrace of corefile (double crashed memory dump)
Thus, sys-assert does not handle SIGSEGV.

Change-Id: I826ee2bab796fc87b2b4a003683315514af96339

Fix handle leak

Fix handle leak of file pointer at crash-stack

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

Merge "crash-stack: use libunwind for i686 and x86_64" into tizen_3.0

Merge "crash-stack: fixed attaching to thread" into tizen_3.0

crash-stack: use libunwind for i686 and x86_64

Change-Id: I0022cc3ac772ba26ed67f7d64d45c3ab4e22db88

crash-stack: fixed attaching to thread

It is not necessary to attach to parent process main thread to
resolve stack in another thread. This commit also solves hanging up
on 'D' (and 'Z') processes - main thread is not attachable,
when a crash occurs in another thread.

Change-Id: Ic3a085951acb4057fcdd4a476693954d1d3a522b

crash-manager: consider backward compatibility for crash-stack

Though the crash-stack is available about tid option,
previous kernel has not supported this feature.
Therefore the crash-manager should call crash-stack
considering backward compatibility.

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

crash-stack: deprecate corefile option

crash-stack should be able to generate call stack
using pid only.
(Generating corefile causes sluggish issue)

Change-Id: Ie9e74494b966838d1dd6548424d51bfd2df4c1e0

Revert "engineer mode: make .debugmode on engineer image"

.debugmode is moved to other package

This reverts commit b153665a12cda6a29f9ba9d12db7f2682ade9af5.

Change-Id: Idfe9cae63b8f4e4122c74abb144250eea45d9f07

crash-stack: get registers from tid instead of pid

Change-Id: Id4ea3f57808420ec6ac0941c2612a5af51136c96

crash-stack: work around hanging up on 'D' processes

Processes in uninterruptible sleep state ('D' status in ps)
cannot be ptraced until they leave the state. To prevent
hanging up of crash-stack, we check the target process state.
If it is 'D', then we give up ptracing it, and instead
print the information from /proc/<pid>/wchan, /proc/<pid>/syscall
and /proc/<pid>/stack.

Change-Id: I82644051a5499c42db2a8e51a7af98693e8da363

crash-stack: Drop duplicate call stack footer

Change-Id: I938f2ead95d312450edb853740cce47becf796fc

crash-stack: print memory information

Print information from /proc/meminfo and /proc/PID/status

Change-Id: I6c66e9ee6c8eca20a8925e9ecc23e09c0dabcb25

crash-stack: change error reporting file handle

Change-Id: I734b431f56f9e32c440839e1bd4e5b90112773ec

crash-stack: print maps information

Print information about mapped memory regions.

Add stub for printing system and process VM information.

Change-Id: I47b2c5f837620f9d22b3270a75d8f5e663d44c78

crash-stack: print executable file path

Change-Id: Ib5946bcfb5bbad8af7b7bbfc7c8568a3cb3ab207

crash-stack: support for resolving stack from threads

Change-Id: Ic80b6b83acb80b3c332b6b6674fbf662b1003586

crash-stack: put more effort into getting names

It seems libdwfl-0.153 sometimes can't get symbol names
automatically. This commit adds fallback procedure, that
takes module name and mapping provided by libdwfl,
and - using libelf - looks for the symbol name, instead
of relying only on libdwfl.

Change-Id: I7e65f810b46dc43bea217b715852a5ae5c3d70c9

crash-stack: reformat callstack output

Change-Id: Ica727d63de389d8a4a2f1ff7cfced2a463630282

Merge "crash-stack: resolving symbols extracted from printing function" into tizen_3.0

Merge "crash-stack: print information about threads" into tizen_3.0

crash-stack: resolving symbols extracted from printing function

Change-Id: I0ae01bc7a2a9a0e6f084a9e275e1025058645c77

Merge "crash-manager: pass tid number to identify failing thread" into tizen_3.0

Merge "tests: Add a test program for crashing with threads" into tizen_3.0

Merge "crash-stack: align output format with sys-assert" into tizen_3.0

crash-stack: print information about threads

Change-Id: Ib0e2d9c2bdbd5fd4eacc0608a8c5ecccdfd4f2a8

crash-manager: pass tid number to identify failing thread

Accept tid as second argument for crash-manager and pass it to
crash-stack.

Change-Id: Icfc0db6f5675eb781777ea7a7c5dc2d4cb581105

tests: Add a test program for crashing with threads

Change-Id: I0b9ef8567b0bb1203fdd9fe3e0aa753e25ff50df

crash-manager: make secure program

- Use secure function (strncat)
- Prevent buffer overrun

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

crash-stack: align output format with sys-assert

Make crash-stack print register information the very same way as
sys-assert does it.

Change output format slightly for 64 bit architectures to match hw
requirements.

Support both i686 and x86_64 in the same architecture specific file.

Change-Id: Ica03360b61c1aa133b83c5622334cf4f375db3c2

Update authorship information

Change-Id: I064948790ce22c0167ce6986a4d0a0e78fd974ab

crash-stack: separate libunw code

Change-Id: I1f24781316a78439db862c3ab0d2492fea095c1c

crash-stack: add symbol resolution for arm

Change-Id: Iabb556a96907156a9b8775cdb25281dea560ce69

use libunwind for ARM in case of McTernan unwinder

Change-Id: I6fb6f95df6f09a1b4c7e38489606aaab1e452e59

Resolve crash-stack build issue in emulator

32 bit: Add stub function for _crash_stack_print_regs
64 bit: correct output format specifier

Change-Id: I4d95ab2ebee9c86a4fa05015f6ca5b3ccc8ea151

Revert "crash-pipe: read and parse core"

This reverts commit b3661520e5b660ce7352381e6bfa5d25647357e4.

Change-Id: I01557e7b7bf4ed5f8c278e2e168d8aaa04f8d49e

crash-manager: change default configuration values to unlimited

For ease of debugging, default values of coredump configuration
are changed to 0 (unlimited).

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

tests: add a glib based crash test

Change-Id: Ib5c2a176e0108b1fa20cbd0d156fb1f29139b8fa

tests: add a simple test

test1-crash crashes and dumps a core. test1-sleep enters an infinite
sleep enabling ptrace-based stack examination. test1-ill triggers
the "illegal instruction" exception (SIGILL).

Change-Id: I995f5ce8817e49ec57bac6cac8e7f458807d03f0

crash-pipe: read and parse core

Read basic information from elf notes segment and dump:

+ NT_SIGINFO: siginfo_t describing the signal that killed the process
+ NT_AUXV: auxiliary vector
+ NT_FILE: memory mapped files
+ NT_PRSTATUS: registers

The information is at the beginnig of the core (< 8kB) so it is not
necessary to read large amounts of data.

Change-Id: Ia2232ce88bb039e49f2129f7a28ff1f4a7c9bbda

crash-pipe: Print maps information

Change-Id: Idab48c5dde20e0aa2a8337b7008a8244a612d844

crash-pipe: Print memory information

The code has been ported from sys-assert.

Change-Id: I64ae2fb0d2a12f7def29328b1ac46cb6c335e0a5

crash-pipe: Print the executable file path

Change-Id: I730978b87b15a02e9464505426d53b9d2037d85f

crash-pipe: the buffer does not need to be a static variable

Change-Id: Ibb1f489564ecc888451d59c549db9dab9d0d97e9

crash-stack: report on siginfo_t

Change-Id: I86e8058eded47f23c51d213fffb56f2f078931bc

crash-stack: Print signal information

Change-Id: Ia6d535d1d229dbc55aafd6b50529062e63b411ff

crash-stack: link against libebl conditionally

Link against libebl only if crash-stack is built with support for
core dump files.

Rewrite conditional building of sys_assert to use %bcond_with.

Change-Id: I50edbd417a5426a43da3b9c0e912933a7db95f4b

crash-stack: Print register values

Change-Id: If8d82aa32eb524ffb23715af6016f0fec12bd144

Add .gitignore and .dir-locals.el files for easier development.

Change-Id: I6ecab2063adb5d5660592becc31db8f21e67c89a

Fixed coding style & added doxygen docs

This fixes and adds again commit 11c142a267540ee070db0c8c1dc47c022edd02a6.
Thus, this reverts commit 7ec36fc1820b0b3622f0c1d941439e906a4603ff
which reverted commit 11c142a267540ee070db0c8c1dc47c022edd02a6.

Change-Id: I944d1e9a664cbc38e84853f9a67b093ecb68754b

crash-stack: adding fallback heuristic method for unwinding

This reverts commit b7ef1fff4dd5f8f754df1431fe4b80fc1901b459,
which reverted commit f29763ccb1cf24bea7807b55ecd9722e3cb03743.

In case no better method is ready or available, this method
may yield some useful results. It simply walks over data stack
and checks word after word if it looks like an address from
any function. All such addresses are considered as call stack
functions. It should generally show all the called functions
that left any trace on the stack. However, among those there
may be a number of false positives.

This is currently used for x86_64 architecture.

Change-Id: If8fd88b8ab3e345a256006127c36bcb4022cc736

crash-stack: unwinding by frame pointer on aarch64

This fixes and adds again commit 269790304eea1bde27644962efa7803c7ab611df.
Thus, this reverts commit ba3f2151f97c3e4bba1ac04c953f28ac4eb1e463,
which reverted commit 269790304eea1bde27644962efa7803c7ab611df.

To unwind call stack on aarch64 we need to use external
method, as libelf 0.153 does not support unwinding yet.
Possible methods are:
- using libunwind;
- manual walk with frame pointers;
- heuristic unwind by inspecting data stack.

This patch adds support for unwinding on aarch64 with frame pointers,
along with changes needed to modularize unwinding.

Change-Id: I461a06c96d56804fefb7167550e44074e734c94b

crash-manager: refine internal code

1) Use flock instead of manual lock
  The standard function flock substituted lock function
 implemented by itself.
2) Clean dump at once
  The clean dump logic can be done in single scan.
 It would simplify the operation of the function.

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

Change log buffer to system

Use SLOGs instead of LOGs

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

Change compressing format to zip

Change compressing format from tar.gz to zip
due to compatibility issues

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