crash-stack: Drop unused get_threads()

Change-Id: I8146fe9c52889d872967dadfa12065dabb42cc3f

Release 6.5.13

Change-Id: Iccc7f0dd4aafcf6056282a71880b4dd8df89a3a1

Allow to get bugreports using the diagnostics API

bugreport-service registers as a org.tizen.bugreport-service and allows
to get bugreport and crash-info report by dumpsys.
For example: to get all bugreports created in the last hour run:

  dumpsys org.tizen.bugreport-service -- --type bugreport --last 3600

Supported parameters:

  --type <bugreport|crash-info>  Get specified report
  --last <sec>          Get reports created in the last <sec>
  --from <timestamp>    Get reports created after a specified time
  --to <timestamp>      Get reports created before a specified time

Change-Id: I0d3e04c34b63f3a559c8bde9af703eeabeb4e7ae

Send diagnostics event (BugreportCreated) when new bugreport is created

Change-Id: I0fe77932bec656e1abbf0a1fd79eb68ae233ea3d

libbugreport: Drop needless -service suffix

Change-Id: I1f7c0d4b021c5bee76d2325649c55bdc6478eac1

spec: Fix crash_root directory permission

Change-Id: Id2f1f4ac5fb2f160011d5ccb1976645f8b853d55

spec: Ensure needed crash_worker user is available during package installation

Some system users are created by security-config package %post script, thus
we need to ensure this package is installed and configured before we install
packages containing, in our case, crash_worker-user owned files/dirs.

Change-Id: I122e992936f4569cbb5f42f0f38f4ce9f5fb2100

Move the libbugreport library to the separate spec

A separate spec is needed because diagnostics depends on libbugreport,
while bugreport-service requires diagnostics.

Change-Id: I809e21be045ca6d29d5851032d135a308e816c16

Release 6.5.12

* Add new dumpsystemstate-service
* Rename crash-service to bugreport-service
* Decouple bugreport-service from crash-worker dependencies
* Rework packaging to make it possible to complile out/disable crash-worker/livedumper/etc.

Change-Id: I7b3bdd9156e786d7bbba1d49ef6db36659eb7c48

spec: Santize programs' permissions

Use default only where necessary and clearly
write the reason behind it.

Change-Id: I96b696c4b8268a9548be3f105dc827a67b436742

spec: Provide libcrash-service to ease transition to libbugreport

Change-Id: I33d83f9d5e40e6d6d1940597424bba87fcc4552d

packaging: Package only bugreport-service and dumpsystemstate-service for TV

Change-Id: Ia2b330a71d6775821dce0db5f2076074dfe396c1

spec: redesign

Change-Id: I5b2b2337413c4ea019be4c876449263d8ef72257

bugreport-service: Independence from the crash-manager

The changes that this commit makes allow to install the
bugreport-service without having to install crash-manager.

Change-Id: If448436c0dff6a69f1bef6b4d249c199f549dec8

libcrash-service: Do not export unneeded symbols

Change-Id: I5584761b83f386568b869be921f667d18c6cfd3e

bugreport: Ensure library is linked to libraries we actually need

We always link to libcrash-manager but it might not be needed,
if livecoredump suport is disabled.

This commit prepares bugreport-service to be libcrash-manager-independent.

Change-Id: I4d506e517ce2da22daa047741187a15f91da4180

Add dump_systemstate-service

The service registers with the name org.tizen.systemstate and provides
the output of defined commands via dumpsys interface. The registration
of commands is done by adding dumpsyscmd parameter to the extra program
configuration file. For example, the following configuration in
/etc/dump_systemstate.conf.d/programs/crash-worker-programs.conf:

  ...

  [buxton system]
  order=141
  cmdflag=b
  title=System configuration
  path=/usr/bin/buxton2ctl
  args=dump system
  dumpsyscmd=buxton_system
  allowdumpsysargs=yes
  ...

allows to get the output of the command:

  /usr/bin/buxton2ctl dump system

by calling:

  dumpsys org.tizen.systemstate buxton_system

The allowdumpsysargs parameter is responsible for the possibility to
provide custom parameters via dumpsys (disallowed by default).

Change-Id: Iff9569d4975ab97addfcf32ea6c22d7237275b35

dump_systemstate: Extract the function to read ini files

Change-Id: I53b79be8953bfc1d6c3c420913d8afa47cac5004

packaging: Split packages to allow installing only bugreport-service with its deps

Change-Id: I8e225cc465b6a6585fd7f3f292e249a27bcff898

bugreport: Ensure DLog tag is always specified

Change-Id: I4ccde9340c18e9589fdfdc03ba0937c2632ff4bf

Change the crash-service to the bugreport-service

Change-Id: Ieea6c499a279f8bbe6c61232bbc49c4f0f768691

Create a report with only dump_systemstate output for PID <= 0

Passing PID <= 0 to livedump_pid() API method will create a report
containing only dump_systemstate output.

Change-Id: I38dd566acb0ff2f1a0ecf0a492dbdac91343c215

Release 6.5.11

Change-Id: Ied63aff39a5c9b5d03874683e9e2ab08bab6c661

Re-create crash_path for compatibilities.

The crash_path should be installed at image creation time for several Tizen testing infras.
Until Tizen testing infras will be modified, the crash_path should be remained.

Change-Id: Iea7166e87a2dbd3b55bb95f8d8a492d17dec50a0

Release 6.5.10

Change-Id: I1a7060c4c3949a28f2e7d66c31ff85063e3eb295

Set permissions explicitly for directories and reports

Change-Id: If01ddf2a7f73a6b808f3cd8d39f20a77c74231d3

crash-manager, service: Set a restrictive umask

Setting such a mask will make any files created on ocassion (such as
temporary files) unreadable by others.

Change-Id: I6843a49cb123a663effb1ab99e8118e968786624

packaging: Drop redundant directories

These directories are created on-demand by crash-manager/service.

There is no need to main correct permission/label set in multiple
places.

Change-Id: Ie81cd358d9e17d4249bbdc0a2ed791b5097fa583

Merge branch 'tizen_6.0' into tizen

Change-Id: I3d8551cda3f54fd4bfa617667a54ef7238c39949

Release 6.0.30

Change-Id: I5480be318f6855562e7b4599be2b002d5d97ca37

diagnostics-run.conf: fix tmpfile location in 64 bit

Change-Id: Ieda58ed71f73503127ba44b85f8bc0d6fa6e26f0

crash-manager: Set primary group

Previously only supplementary groups were set, leaving main gid to
be 'root'.

Change-Id: I50494ca40192c985d07cf07827c4f44c4f99b1d7

Merge branch 'tizen' into tizen_6.0

Change-Id: Ib47738a925b7ce8f241d1f865a4ca6767013d273

Release 6.5.8

Change-Id: I4a4549b63a6ea415d8a1818d7bb1b782bebe6eea

crash-service: Drop priv_* group from policy as these are not allowed by tizen sanity checkers

It seems priv_* groups can not be explicitly listed, as image creation fails with:

[01/21 15:50:01 UTC] FAILED(assert) at /busconfig/policy[3] : Group 'priv_*' is not allowed.
[01/21 15:50:01 UTC] Found 1 Fails
[01/21 15:50:01 UTC] Remove conf file : mv /etc/dbus-1/system.d/crash-service.conf /etc/dbus-1/system.d/crash-service.conf.fail

Change-Id: Ieea004890dfbccddf9d16ee949f55f3a02461bcc

Merge branch 'tizen' into tizen_6.0

Change-Id: Iaec2734d534c06682dc3bb85b7339fc6a685e14f

Release 6.5.7

Change-Id: Iab075746a882ac423c212f4b803fa8fa48984dfe

crash-service: Re-add privilege <check> for livedump

dbus-daemon uses getgrouplist() to get group membership, meaning
that we can't depend on group membership alone.  For this to work
dbus-daemon would need to use SO_PEERGROUP, which was introduced
in Linux kernel 4.13.

For now we have to keep both checks (<group> and <check privilege..)

Change-Id: I86b7664f19292be71c5d7c2ca27f837ec7799a96

Release 6.5.6

Change-Id: I843c2c7021a04b42799cb5f2b4931fe1a1145fe7

crash-service: Simplify the diagnostics get_file access control

Root is impicitly allowed by <check ..> directive, there is no need
for separate section for this.

Change-Id: I8ff19674fd2610ab1255db14658ae0e1ceeab5e1

crash-service: Permit service & app livedump API clients

The d-bus livedump_pid API can be used by both services and apps.

For service to use the API it should be member of (supplementary)
priv_livecoredumper group.

For application to use the API it should be given the
"http://tizen.org/privilege/internal/livecoredump" privilege.
However, it's internal privilege so it can't be given from the
application manifest. See 'livedumper' section in NOTES file
for hints.

Change-Id: I01586c2c00f7a15d2553e4284ce373a03f9433b1

Add information about the need to manually add a privilege

Change-Id: I9832f5fb566e183e414e1fc0f27d3275f2931aa8

Merge changes I4fbe1307,I4ed60496 into tizen_6.0

* changes:
  Release 6.0.27
  Fix dbus policy for livedump_pid

Release 6.5.5

Change-Id: I78053a4c9608f279cb0f71daf72740bd54103943

Release 6.0.27

Change-Id: I4fbe1307c1ce48917057b19692a53676d48a344c

Fix dbus policy for livedump_pid

Change-Id: I4ed60496efefc70ef2b8fb2b8590e4f430e016cb

Fix dbus policy for livedump_pid

Change-Id: I4ed60496efefc70ef2b8fb2b8590e4f430e016cb

Release 6.5.4

Change-Id: Ice6551ac00608f994627d5f5c85a3f15457bc17d

filename change from "corrupted_cmdline" to "nonascii_cmdline"

isgraph cannot check cmdline with unicode.
Thus, it will be better to change to nonascii_cmdline

Change-Id: I60c166512cc0c7cb766d0223f71b336403aa644b

Release 6.5.3

This release introduces cmdline santizing, to avoid creating reports
with 'strange' characters in name (as it's based on cmdline).

Change-Id: I2eb83c0ceb391608db9388115260b9464ecb9c90

crash-manager: Santize cmd_line early

There is no value in passing cmd_line that is potentially corrupted.

Previously, we wanted to share original cmdline value via dbus-notify
but this no longer seems correct as we have to replace cmd_line
completely when it contains invalid/non-printable characters.

This patch changes behaviour so that santized cmd_line is available
universally via cinfo->cmd_line.

Change-Id: I9a54781614d89c6abb89ef0c0755d8a23a26f55d

crash-manager: Create reports with `corrupted_process' prefix when cmdline is corrupted

Change-Id: I62cd86e70d4d1e347bdadd4d2a2a4fcb448b88d4

Release 6.5.2

Change-Id: Ib9fea66140376d0e4b0a0fe6b93b02b639aa8229

Fix SVACE warnings

Change-Id: I7ef2563bee2bf1690b2a38554cc15bf0e10f92fc

Fix null dereferences

Change-Id: Id23b9aaad60752f797783f9ccc6f9bbfa9357941
Signed-off-by: Michal Bloch <m.bloch@samsung.com>
(cherry picked from commit a9a8f5e3955f55e03a05878a381257085dc02951)

Fix null dereferences

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

Merge branch 'tizen' into tizen_6.0

Change-Id: I5be0529ee820cfb1cedd987686b954b522209cc7

Release 6.5.1

Change-Id: I9e72706205c738df27cdc73de7b46797e5157018

Release 6.0.25

Change-Id: Id7d2a7d722ce8b0ee94fc1bf5db390ba52928cc3

Add a SHA1 hashsum to the so_info file

Change-Id: I2508c06f58a9385973511e86737aa6b146fe2de9

Add VCS to the so_info file

VCS is read from the rpm database

Change-Id: Ie8359f1ea91bdff9c77517b8d709334ec0e6226e

Release 6.0.24

Change-Id: Ibdd55db0abcde2a91ea08a40dca96200dfede63d

Update the crash-manager upgrade script

Change-Id: Ibd9fe951012a828a29038f1ed06c797f4fbecb19

Fix dbus_notify system test and return the result of tests

Change-Id: I1fa460e024bcb7eeba0c9bdde42903edb91dd8a4

Add a NULL check

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

Fix incorrect variable qualification

Change-Id: Icf878243cbc7dbd89ac261d9c55108296738d7fc

Merge "Use machine-friendly syntax" into tizen

Fix some file descriptor leaks

Change-Id: I1bb8cc2580ca208cc8387a84528d9a60e6909cd3

Use machine-friendly syntax

Various tools (compiler, static analysis) might see better with this one.

Change-Id: I6567c41ca6c69cb026771ce7e7a802efb578d2f9

Drop elfutils remains

These were not used anyway since own parser has been deployed.

Change-Id: I7c961ea8dd05059944f2a6dcae4ae63c305a8b18

Release 6.0.23

Change-Id: Ie93e2adbab1653ce14e3ac77e3a98afe3e7591ec

Fix coverity issue - untrusted divisor

Change-Id: I85689dc3753e24ebec6ab88cd6e0e39f57cce442

Release 6.0.22

Change-Id: If8974ef1ca38a83fea8e571e3058e12af7c03d96

Calculate the call stack hash sum

The hash is used to identify crash reports that contain the same call
stack.

Change-Id: Ib08e6775d3f3c671b660d92449de78c0d3bb74ec

Release 6.0.21

Change-Id: I394cebd522fec96a52207f26869850ec18fcb595

Fix printing the callstack in case the module name cannot be found

Change-Id: Ie077c4a53467996d63a71ba8cfe6f58331a4ff16

Fix building on x86

Change-Id: I99a1b7ad017c5f745cf2304a333d1f130c251586

Release 6.0.20

Change-Id: I73f6b4b1b66b4a3a37689759c1c568c742698676

crash-manager: Change Smack lablel to System::Privileged

This partially reverts 8e2f5635896 ("Adjust smack label").

The System::Privileged label is needed again because on
newer kernels (>= 4.20) it's not possible to read/ptrace
processes listed in onlycap set.  Practically, that meant
that many of Tizen processes were not able to generate
coredump properly.

Change-Id: Ifcdeceb88cede67a0d8192474698addfc9aa338b

Move crash-worker tests to the /usr/libexec directory

Change-Id: Ib4daa77836316318e612740871e4de982d5357cd

Fix building on aarch64, after changing the ELF parser.

Change-Id: I41557a2149ca4b6c5d89ab16a89199d1cdf33170

Read ESP and EIP from registers instead of /proc/<pid>/stat

Since linux kernel commit 0a1eb2d474edfe75466be6b4677ad84e5e8ca3f5,
/proc/<pid>/stat reports EIP and ESP values only if the process is core
dumping. So for livedumper to work, these values must be read by
ptrace().

Change-Id: Ie748e694fc1ae30841c2fbb709fbd9abe67185ba

crash-manager: Always emit new style ProcessCrashedEx

ProcessCrashed signature varies between Tizen forks, so introduce
ProcessCrashedEx which does not have this problem.

Change-Id: I4cf3f2327690d6bedb6d39dc85ca96487e859acb

Check for existence of the ExtraScript

In the case where ExtraScript points to non-existent script, the crash worker
will create child process which will remain zombie until it's reaped.  The
reaping is handled after dump_systemstate finishes.  This will cause zombie
process to exist for duration of dump_systemstate execution.

This change prevents creating child process if ExtraScripts points to path
which is not executable.

Change-Id: I4caba47812b6990e9b5b54fe9929930cc667f4eb

Add crash-service part of the diagnostic API

Change-Id: I869ebed29b6657a2227fae407079c643819a551b

Change the way ELF file are parsed

Change-Id: I737ec17f8a5a6e71efe24f7f44ad1e08775d24b7

Add an ELF file parser

Change-Id: Ib71d0de73a6dde20f5ccfca1a8b8f0d4e283a19a

Release 6.0.19

The most important change in this release is the addition of the INFO
report in JSON format (*.info.json)

Change-Id: I5372c7eff95940241365abe8beacb6dfc71245a7

Change the function signature to a more universal one

Change-Id: I3fea24d1448ffba3d6ff24e7c2090e6d7e0565e0

Add tool to convert info.json report

Change-Id: I4c10ef5880ae919c8b4bf7a8167344db6f8ed2c7

Add report file in JSON format

Change-Id: Ie61fcf83ebd6c8dd536e54e4206094d2d3b94d67

Simplifies the printing of registers

Change-Id: Ibf6b17258bbb6e108f896db5ca3f4b50db7dde36

Change getpwnam to getpwnam_r

Change-Id: I1876f1f16d205698f3385cf472505bcab83de88e

Fix memory leak

Change-Id: I472d42d4f486bcad970b52d881c295edd8ec2d49

Release 6.0.18

This release brings option that allows to specify whether the process is
to be released as soon as all the necessary data has been read or only
after the entire report has been created.

Change-Id: Icb2cab7884966505a26c97aa86745721252b6bb7

Fix 'full_core' system test

Change-Id: I9bd5831f134141546fe435b9042b87a13ed39c1b

Add option to release the crashed process as soon as possible

Change-Id: Idd951c4b8c50ba1e29524339358475acdbacd382

Merge "Read the /proc/<PID>/comm earlier" into tizen

Read the /proc/<PID>/comm earlier

Change-Id: Ied692bc4e770c9aa3b8f03f7a2515214bc40cb63

Clarify logs related to coredump handling

The coredump should not be created when minicoredumper (-s) works
correctly.  Make this clear in code and logs.

Change-Id: I2e4e160a46ac4be25143f310ffc336b121530cc8