crash-manager: Fix memory leak

Change-Id: I022a378b011f1f7220f87946c85eca2e7ec8553c

Test: remove duplicated link option

Change-Id: I38948f1902482922e5876574bfb42091cce98713

Release 6.5.19

Change-Id: I164d956ca37d4d87dccc1db74039cc022796702b

crash-manager: Provide TID as optional value of an -i option

Since %i can only be specified in core_pattern since kernel 3.18,
crash-manager must handle the situation where no value is specified for
this parameter. This is implemented in this commit, but the value for
the -i parameter must be specified without spaces.

Change-Id: I2cc2ff5d527a638c0f52df180c3875daf6208e02

crash-manager: Fix TID finding

To find a thread ID that crashed crash-worker checked if
/proc/<pid>/tasks/<pid>/wchan contains pipe_wait or do_coredump.
Additionally it should check if pipe_write is there, because since
kernel 5.10 this content is for the thread that crashed.

Change-Id: Ic45913306d4f5b39ae704abdd58bbd79fef5e7cd

Release 6.5.18

Change-Id: I8c73d1521e36364090d4ac42fd08cc9bc59a1ceb

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

This reverts commit 15e78af58e90069dff13201dfcc89035664ba650.

Change-Id: Id7951c0beaa7322eb3db5e75865e6bb9f601312e

bugreport-service: Get bugreport based on event data

A bugreport identifier is sent along with BugreportCreated event so that
client can request the details of the bugreport to which the event
related:

  static void bugreport_created_cb(diagnostics_ctx_h ctx, void *user_data)
  {
      ...
      const char *params[] = {"--type", "crash-info-json"};
      diagnostics_data_h data;
      diagnostics_get_data(ctx, params, ARRAY_SIZE(params), &data);

      size_t bytes;
      char buff[1024];
      diagnostics_data_read(data, buff, sizeof(buff), 0, &bytes);
      ...
  }

  int main()
  {
      ...
      diagnostics_set_notification_cb(bugreport_created_cb, NULL);
      diagnostics_subscribe_event("BugreportCreated", "org.tizen.bugreport-service");
      g_main_loop_run(mainloop);
      ...
  }

Change-Id: I056f16743e656504dcefe0972eae00bc4b6f58ab

bugreport-service: Add the ability to get coredump file from a report

This commit adds two more report types that can be obtained:

  coredumptar  - tared coredump file
  fullreport   - zipped report

New types can only be used with a specified report, the identifier of
which must be given as the call argument.

Change-Id: I5c769224c5f5ef89c5d1cc5070a7a4af8a7e0b8a

bugreport-service: Add crash-info-json report type

Change-Id: I7a3a1878085e64852fe19ca44a1045de6f71150f

Revert "Fix build error"

This reverts commit 437d159c204e5cae9b5ea6c8a7d294dc73b421af.

Change-Id: I72df3308c18fef162740c783f80c897e31c7a7fc

Release 6.5.17: Fix resource leaks

Change-Id: I3f4b6031baba78e3db2105020db3cbe24f3736e8

Fix build error

[  101s] /home/abuild/rpmbuild/BUILD/crash-worker-6.5.16/tests/system/utils/
libbugreport-servicetest.c:46:51: error: 'CS_ERR_NO_ERROR' undeclared (first use in this function)
[  101s]    46 |  printf("res: %s (%d)\nreport_path: %s\n", res == CS_ERR_NO_ERROR ? "true" ...
[  101s]       |                                                   ^~~~~~~~~~~~~~~

Change-Id: I5f9588ee8c3b9f9be15169a848ac6082e67ce042
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>

Fix memory leak and remove the unused code

Change-Id: Ic746786b2c76314d671a18d992a25451c564197f

Release 6.5.16: Various fixes

 - Restore /opt/usr/share/crash/dump directory for tizen smoke scripts
 - Fix TV build

Change-Id: Id64d47dae3e592089ba1c02c79365af7a19791dc

bugreport-service uses the default crash_root_path when there is no config file

Change-Id: If5d6b0b0323576a27e115a407c3130a2bac2111f

Release 6.5.15: bugreport-service fixes

This release fixes up two important issues with bugreport-service:
 - null-terminate array of getopt args to avoid crash on arg parsing
 - default to 'bugreport' report type when it's not specified

Change-Id: I7e99d19cfacb7f1193314a9dd4c6256b9aa25503

Release 6.5.14

Change-Id: Ib05ad5ca226f68e88d3dd9b842f8232dd359042d

Move the bugreport library to the diagnostics repository

Change-Id: I05ee9089e4fa74fa58f0a8554aa67e82a10372f2

Fix svace issues

Change-Id: Ic4935bd22bccaa3f07f7cbd57055f5f7bfe58ff1

Change livedump_pid() return codes

livedump_pid() returns 0 on success, otherwise a negative error value
TIZEN_ERROR_PERMISSION_DENIED - privilege error
TIZEN_ERROR_NO_SUCH_PROCESS - no such process
TIZEN_ERROR_IO_ERROR - other error

Change-Id: If961771e745e98212ca9ded959dfa504b197f030
Signed-off-by: Konrad Kuchciak <k.kuchciak@samsung.com>

bugreport-service: Add return error code about no process for livedump_pid() method

Change-Id: I9f0182b9b857c4aca685d495f568c1fbd48a8b37

Check dbus bugreport.admin privilege for accessing livedump_pid

Change-Id: Ia41d63e1f461454cb9f9d297113624e96bdb3ca6

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