Free 'hash_str_full' only if it was allocated

Change-Id: Ided519be8f5a6145d0ea018fa37a0f5a7ef7a213
Signed-off-by: Sung-hun Kim <sfoon.kim@samsung.com>

Fix a coverity issue

This patch fixes a coverity issue [1].
Since 'ctx' is dereferenced in EVP_DigestInit_ex call, it should be NULL checked before calling this function.

[1] http://10.113.139.115:8081/reports.htm#v19039/p11006/fileInstanceId=612233125&defectInstanceId=130271535&mergedDefectId=1706287

Change-Id: I0e1e8a7a775734649ecffe9d5adaf0bd71dac8a5
Signed-off-by: Sung-hun Kim <sfoon.kim@samsung.com>

crash-stack: Always print threads information

The default behaviour was to skip thread information when there
was only one (main) thread.  This however, breaks report parsing
as it has been recently added for `bugreport-service --type pid
feature[1].

This commit makes printing thread info unconditional, fixing
bugrepor-service.

[1] dumpsys org.tizen.bugreport-service --type pid

Change-Id: Icfc6ebe3e44c86d241f22cf88f0561f697d7e361

packaging: Add friendly reminder about the need to call set_capability for tools to work

Change-Id: I2c847032505b2fc5c04e3fe3f42a5f2effc3cb50

crash-stack: Check return of EVP_*() calls

Change-Id: Idb2f43986986fd11f696962d17599304dc10c417

diagnostics: Fix errors found via static analysis

Change-Id: Iad3f8e31e62d5a0fe2d87a98e9fdbfe26359a4e0

Change /usr/bin/crash-worker Smack label to floor (_)

This change is needed after kernel started launching processes
using usermodehelper process. Usermodehelper runs with normal
privileges (Smack floor label) all normal policies apply to it.
Thus, in order for it to launch crash-worker it either needs to
have same label or there exists rule that allows given transition.

Adding rule allowing _ to execute System would break Tizen policy
so the other option is to change crash-worker label to _, which
is what this commit does.

There should be no adverse affects of this patch as crash-worker
is already only allowed to be executed by root:crash_worker uids:gids.

Ref: https://lore.kernel.org/all/20220607165003.871993847@linuxfoundation.org/

Change-Id: I805d263fec72b491d0fc0915413d2c3bdded91ab
Reported-by: Sunghun Kim <sfoon.kim@samsung.com>

Upgrade OpenSSL version

Change-Id: I2e1bec24859056fa4c64b096c748b85c592050c4

dump_systemstate-service: Fix possible memory leaks

The `strdup` call uses `malloc` internally for allocating memory
area. Thus, the result of `strdup` should be freed in the
allocation failed cases.

Change-Id: I01a1f1653cd1ddbac5e89cdfb085e0d273414c6c
Signed-off-by: Sung-hun Kim <sfoon.kim@samsung.com>

dump_systemstate-service: Fix a build error on GCC-13

Building on GCC-13 stops on the below build error:

[   16s] [ 64%] Built target test1-default-sleep
[   17s] /home/abuild/rpmbuild/BUILD/crash-worker-7.0.1/src/dump_systemstate-service/dump_systemstate-service.c: In function 'split':
[   17s] /home/abuild/rpmbuild/BUILD/crash-worker-7.0.1/src/dump_systemstate-service/dump_systemstate-service.c:101:25: error: pointer 'result' may be used after 'realloc' [-Werror=use-after-free]
[   17s]   101 |                         free(result);
[   17s]       |                         ^~~~~~~~~~~~
[   17s] /home/abuild/rpmbuild/BUILD/crash-worker-7.0.1/src/dump_systemstate-service/dump_systemstate-service.c:92:37: note: call to 'realloc' here
[   17s]    92 |                 char **result_tmp = realloc(result, (*size + 1) * sizeof(char *));
[   17s]       |                                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Since the memory area pointed by `result` can be freed by `realloc`,
`result` should be freed conditionally.

Change-Id: I09ee12f83073665ef37788fa948b72ce3883ea3f
Signed-off-by: Sung-hun Kim <sfoon.kim@samsung.com>

bugreport-service: Add PID information as a new type of bugreport

To prevent unlimited re-executing a crashed app, application manager
(e.g., AMD) needs an information of PID for the crashed app. But it
don't need the full report of it. So, we need to add a method to get
only the PID while using same diagnostics API call for convenience.

At the perspective of caller, it just uses the same diagnostics API
with an argument "PID" to get the PID of the crashed app.

Change-Id: I165ea2e33840bd803ae4319137bb44f7b2c883d3
Signed-off-by: Sung-hun Kim <sfoon.kim@samsung.com>

Fix build issue on riscv

Change-Id: I37083fc604170613f0d7033afc6fa4cfb12901ae

Fix build issue on armv7l related to ptrace()

Change-Id: Ia33eefb96b7f98c9c983f236d30c6dc643b88d1a

Fix SVACE issuses

Change-Id: I371e7a235ce5502d8a5f01cdfecac509efac7cc9

Add support for RISC-V

Change-Id: Id1cbadf09b55f7da416833981b0e14ce6202cd7e

Fix realpath(3) memory leak

Change-Id: Ic2e2264a8dab194884decfa5acb9035ea712b149

Release 7.0.1

Change-Id: I4d0e812752f4d396668bbca7da1b6222c3261dff

Fix the release of ELF data resources

Change-Id: I66810a17c4fa04db981a6c325e6a054a334d0bfb

Release 7.0.0

Change-Id: Ibb09b26050346252f4c8b7caeffbb0c804a60175

Remove 500.crash-manager-upgrade.sh

This script prepares directories and permissions when upgrading from
Tizen 5.5 to 6.0. Therefore it is no longer needed.

Change-Id: I1c4b4baeea02c4b0c9ac4131da297352b72399ce

Release 6.5.22

Change-Id: Ib867761ec1c4089efd85f8e5f94a74f932d21683

packaging: Drop unneeded BuildRequires

Not only it was in wrong section but there is already
BuildRequires: pkgconfig(diagnostics) in top section.

Change-Id: I66dac6e85a82f350c56675bd370c68a586d299bf

packaging: Update license specifications after relicensing parts of codebase

Additionally, rename LICENSE to LICENSE.Apache-2.0 for clarity.

Change-Id: If2eb9621ad0808970521d64a5dfe8099913ec2b3

Add missing license header to crash-json2info

Change-Id: Id9d16738ab457a78cba9192deb64b0640d39946c

Relicense crash-manager to MIT

This includes relicensing:
 - crash-manager - the standalone program, which uses libcrash-manager
 - libcrash-manager - library that contains most of the logic

After this change libcrash-manager can be safely combined with GPL-2.0
binaries.

Change-Id: I564837e3ffe7ae5291b8e92bf1199f6277e04017

system-tests: Fix behavior when dump directory is missing

Since crash-manager is responsible for creating dump directory
(commit 96a11f21e013e1de3d02ee438c4291d55de56e9e), the
absence of this directory cannot be considered as an error.

Change-Id: I8db05869c951b3d9b893a223d020dc146ba9455f

Release 6.5.21

Change-Id: I8d58aab1e533cb1563d9f053df79a73ea72e8963

Merge "telf: Fix uninitialized pointer read" into tizen

telf: Fix uninitialized pointer read

Change-Id: I8bf8dbed9af82c895ec3040eeafdb03a0d74fd04

bugreport-service: fix memory leak

Change-Id: I8609072ca5a0eaf55c613390ece3d58b891e6d51

Release 6.5.20

Change-Id: I6f2ecd6a3c5ac242ae6e8783c528dd4a76cad0d5

Merge "Continue creating a report even if an error has occured" into tizen

Continue creating a report even if an error has occured

If an error occurs during minicoredump or dumpsystemstate creation,
report creation is not aborted to preserve as much information about the
crash as possible.

Change-Id: I133eae76335fb0a3a2f6d676393a23576493a44c

crash-stack: Drop dead code

Change-Id: Ib463525ae36cd1db3910748e3246e327dc13533f

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