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

Release 6.0.17

This release brings fixes

Change-Id: Iebcb0ec695edf3c589507c9d9a667fac5bb8cab7

Fix: app ids repeats in app's manifest xmls search logic

Change-Id: I3fe0ca9cfef80077c9d3abb40fb439fc0bb87203

Fix Coverity issue

dirname() could return pointer to the local variable

Change-Id: I886df1d2abf73d49f20c11e7ed5eef4595d1c6c5

Check if the file operation failed by lack of disk space

Change-Id: I3921e4631d9d2a84110af0681777bae15e62cfbc

Merge "Returns unique file paths from the maps file" into tizen

Returns unique file paths from the maps file

This prevents duplicate entries in the *.so_info file

Change-Id: I49c4d4949d4d0075b056891557fe027d82b017a0

Release 6.0.16

This release brings following changes:
 - Include packages in the *.so_info file that are associated with
   *.ni.dll
 - Copy application manifest xml

Change-Id: I12bf5e8e227f04e4d6b4714abe56c9944c508545

Add app's manifest xmls search logic

Change-Id: I585cdc6e395efda52390c6059989d4da6afe5edb

Include packages in so_info file that are associated with *.ni.dll

Change-Id: I114a866da9fc70915b616b39869d23cb9e7f22c7

Remove duplicate g_object_unref

Reference:
 https://developer.gnome.org/gio/stable/GDBusConnection.html#g-dbus-connection-call-sync
 If the parameters GVariant is floating, it is consumed. This allows convenient 'inline' use of g_variant_new(), e.g.:

Change-Id: Icacb1a3e8d738a990fa498815bbadfe72bdf2349

Release 6.0.15

This release brings following changes:
 - tizen-manifest.xml is included in report for tizen applications
 - unused temporary directories are cleaned up when creating new reports
 - shared object info is properly collected for binaries without build id

Change-Id: I955a6296b5fd422d66344c0a8bbda6fba75360db

Use bool instead of int in remove.*() functions

In the modified places the int type was used as a bool, so the function
signatures were changed.

Change-Id: Ic172432c33c06d40c1f71ca8137606d7ea102acf

Move unlocking and removing of the temp dir to a separate function

Change-Id: If38fcfda5737e8d39cb496692ca663e2a3d7c6a2

Add locking and cleaning temporary directory

This commit solves the problem of leaving many temporary directories
that were not deleted if the crash-worker has not finished working
properly.

Change-Id: I415403a2531b6e41ebf20a3b185790d9e21a8ca3

Fix report_basic test

Add a delay to m ake sure that crash-manager has the time to run

Change-Id: I26fd47126d1d8b14ed8da6547f88275e2cbba116

Fix libcrash-service test

Because of second trap, core_pattern didn't have the correct value
restored

Change-Id: I71c6f2c401c293769caa816fe2e767b30578d7b6

crash-manager: Clarify when tizen-manifest.xml is copied

Change-Id: Iba48fc54f9809c077b5ebc0fbe8463186abd767e

crash-manager: Fix free of invalid pointer returned by dirname()

This fixes SVACE issue (overwriting dst_dirpath pointer by dirname(3)
result, which should not be freed).

Change-Id: Iad2ed92c6111f8133e3c6363876ff3fad80557f3

Merge "Add tizen-manifest.xml to crash-reports" into tizen

Fix build id reading when the binary don't have it

Change-Id: I82db306cad980104e096e8b738db24fc5e3547bb

Add tizen-manifest.xml to crash-reports

Change-Id: I6fc7f723ad33cd825e2c5f85b42c7b134a0f1f8b

Release 6.0.14

This release brings /opt/usr reporting to dump_systemstate utility.

Change-Id: I1aada1d58681272703b7c64181d0c7aceccc3d84

dump_systemstate: report ran command faithfully

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

Merge "dump_systemstate: Add /opt/usr disk usage section" into tizen

Merge "dump_systemstate: Use compile-time parameter check in get_disk_used_percent()" into tizen

Merge "Fix cmp_backtraces test" into tizen

Merge "packaging: remove libebl dependency" into tizen

Fix cmp_backtraces test

Change-Id: Ia628adfb80a4945d34e9847e864c903106dba3e6

dump_systemstate: Add /opt/usr disk usage section

Change-Id: Ib52cba397f7272c14dcb7623efc85535d4d98d8e

dump_systemstate: Use compile-time parameter check in get_disk_used_percent()

Guard functions with assert as it's programming error to invoke
it with null path.

Change-Id: I3a312f90c0553c1dd9e9233551c0d42b89c260f7

dump_systemstate: Also list files when reporting /tmp disk usage

Change-Id: I0ea99ca0a05a7d620a4462d6dc44257aa6e4bee0

packaging: remove libebl dependency

libebl is not needed anymore.

Change-Id: If88fa16b361c70c9155e5240a03cbede3e235074
Signed-off-by: Dongkyun Son <dongkyun.s@samsung.com>

Release 6.0.13

This release introduces ability to disable creation of .so_info files.

Change-Id: I6bbc308e0d15b1872e43fd5c5f89142d76b9943d

system tests: Sort entries

Change-Id: I8b5fea6bcd1a9cc50b9efdf96820b444a96d534f

Allow disabling so_info generation via config file

Change-Id: I89d3b687225a7a85498344c9dd84c17ba4f3e33b

Release 6.0.12

Notable features & changes:

 - dump_systemstate: move information about most of the programs to config files
 - dump_systemstate: add cmdflag= and order= support
 - crash-manager, service: add information about C# libraries to package info file (.so_info)
 - whole codebase: reduce default log level to INFO
 - drop separate spec for system-tests

Change-Id: If47fc36e26954309a2c18262f32a69053bab2eb3

dump_systemstate: Allow passing arbitrary flags for cmdflag= usage

Change-Id: I0099e5248f5665af41dbbc22ef17932213f68abd

Save in the *.so_info records related to C# DLL files.

Change-Id: Ibee68bec0fdf3ae07c64b1240093269ce51ec6a6

Copy address of regions related to C# DLL files.

Change-Id: Ic2e7e4b78d0ba85efcb6c3735152c5a5bffb395e

Change the way the process name is reading

Applications launched by process pool have a modified
/proc/<pid>/cmdline, so cmdline must be treated in a specific way.  Now
the process name is the content of /proc/<pid>/cmdline to the first
occurrence of ' ' (space) or '\0' (NULL byte).

Change-Id: I3af1b9d1df86540b16ea238baa0f03152545e408
Signed-off-by: Mateusz Moscicki <m.moscicki2@partner.samsung.com>

Merge "dump_systemstate: Move most of the runnable programs to configuration files" into tizen

dump_systemstate ordering tests

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

dump_systemstate: Move most of the runnable programs to configuration files

Change-Id: I2b68d40de0f4309cc3a25dd6b2786bf5e0ff0902

dump_systemstate config: add `order` config entry

Sort thereby

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

dump_systemstate: Allow extra programs to be turned on/off based on a cmdline flag

Change-Id: I287c46d5286e622c58d772b8063e62f66db82e51

config: Ignore case of ReportType key values

Change-Id: Ifb46e382eaf3b56681103b4220a85a93dd9e9847

Add informative messages for crash-worker processes running with high LogLevel

Change-Id: Id7a6e66d3dd371b0efb961195daca98ef42b5260

log: Drop _SE logs

These are used incosistently. No benefit from _E/_SE logs known.

Change-Id: If33cd93240ddf6d30e07ada9f23342ae309fce4d

log: Make LogLevel statically configurable

Change-Id: I6d38d777ec5aaef0e8ba71d6d1c91907c4a68057

Move system-tests to main .spec

Change-Id: I675555baab83ce3051401757054bde3cf4317271

Release 6.0.11: Hotfix: Do not return pointer to member of destroyed pkgmgr structure

Change-Id: Ibe121a4d98cec1fe4aaaf61dd8b321f0c88ddd9a

Release 6.0.10

This release brings ability to exclude certain paths from
dumping via ExcludePaths config section.

Change-Id: I49bf203e836bf610ebb8836adb9af45cb23cabbb

Introduce list of processes excluded from dumping

This commits adds ability to specify list of executables
to be excluded from dumping via special new ExcludePaths
section, eg.

   /etc/crash-manager.conf.d/local.conf:
   [ExcludePaths]
   NameNotImportant=/usr/bin/daemon-with-sensitive-information

Change-Id: Ia1c7f6570a8fc243458c48cc8f5284ab461be256

util: Refactor get_cmd_info

Change-Id: Ie7c0584487367f7c059c49b0cc309ef5ac7e6542

Janitorial: '<foo>*' -> '<foo> *'

Place pointer signature consistently with rest of codebase.

Change-Id: Ie183a11724fe886036cf2929ca6acdfac989da9b

util: Add function to normalize paths produced by kernel.core_pattern %E

Change-Id: I0afbb7d946d16b2fff293c8697ae2e6d7db53e43

Complain when livecoredumper is needed and not installed

This brings back functionality dropped in 22a0cccdad ("Simplify
struct crash_info setup").

Change-Id: If5ddddee6b99784d7ada9a93e89d3ad975facb6b

config: Be more verbose about consequence of ExtraScript OOM condition

Change-Id: I7add0d5df5fc2a0369c011574eff7a1eccd8343f

Release 6.0.9

This release brings numerous fixes and one change - reports are named
with pkg name prefix for apps (regular processes still are named after
cmdline).

Change-Id: Ia0b6345e5d3735fa7e15cd0621818e55118a11f9

Magic shall not prevail

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

Simplify struct crash_info setup

Change-Id: Ibbb13c4534b89d6e1ee4b951a438ae1748ae30d2

Use package id as prefix for report files

Change-Id: I37ea45e2c0a262e9bcd2bb3bcdb97edba83cbc7d

crash-manager: Be a bit more verbose about consequence of missing TID

Change-Id: If5b5a4c5050c9fc5afea233311de6145222c445f

Drop sys-assert leftovers

Change-Id: Ia220cc73a1b1e60f165aedd58cdfe9360c8de63b

Merge "Adjust codebase for iniparser upgrade (4.x)" into tizen

Release 6.0.8

This release brings minor fixes to dbus policy and system test.

Change-Id: I2bd9cc0a7979dd92e0fe819e0eeca0ec37afb28c

Fix livedump API DBus access policy

This commit partially reverts commit f24c53fb20 ("crash-service:
Restrict allow livedump API via privilege")

As discussed with SRPOL Security, the 'group membership' of given
process does not map directly to process 'having privilege', as
verified by Cynara/security-manager.  For now the checking has to
be done per-user for system services and via-privilege for apps.

Change-Id: I2a7b76174adb22be8ffe55e41678b48938d90bbb

Add missing system test to spec

Change-Id: I7d7b4d2a91e40938e65cb9153ebfc8d0e320876e

Release 6.0.7

This version adds support option to support legacy 'ProcessCrashed' signal.

Change-Id: I964c7f8a58ac96f3055403d9a5c33aee223747e3

Add system test for legacy signal

Change-Id: If4e2d007ebac2de6a11d64137c9d93e5efc75b6d

Add option to support legacy DBus 'ProcessCrashed' signal

Original author: Kun-Hoon Baik <knhoon.baik@samsung.com>
Reworked by: Karol Lewandowski <k.lewandowsk@samsung.com>

Change-Id: I64b950b99d81b2a0a4ba4e67938d865431c1a25e

Adjust codebase for iniparser upgrade (4.x)

Change-Id: I3d8e613b8d1cccbbbf84b7dceac5abde57578fe6

Release 6.0.6

This release brings following changes:

 - Use designated privilege to limit livedump API.

   This replaces temporary use of per-used privilege
   check in dbus policy.

 - Gcc 9.x build fix

Change-Id: I2bad2792af9aefbd210cd1537293823a664d1a8f

crash-service: Restrict allow livedump API via privilege

Use newly created http://tizen.org/privilege/internal/livecoredump
privilege to restrict access to crash-service's livedump dbus
interface.

Change-Id: I9b2f560c061426d561674e3560b60c10e36c384e

crash-manager: Make permission handling in maps more abstract

This also fixes gcc9.2 build error.

Change-Id: Iae099a366063cd69cca840bd83121a8930d89088

Release 6.0.5

This release brings ability to extend configuration via snippets
in /etc/crash-manager.conf.d/ directory.

Change-Id: Ic01f780a2bb3868e48f02bbe95ac725adcb384cd

config: Ensure freed pointers are set to null

Change-Id: I779cff27a6873187e5f17415469864d6ccd0d1b7

config: Search for configuration also under crash-manager.conf.d directory

Change-Id: I9c5680eafb467a0514feec5225cb794609ba1823

config: Separate config loading from processing

Change-Id: Ief38894e03904f9947ee72773f07044280f62b41

crash-service: Stability monitor does not need to own crash-worker's dbus name

Change-Id: Ida73ce27ac6ea040dcffb81f3a61e4096dc81940

Fix file permissions

Change-Id: I078c745cc56e1cf108997f2e76897dbe4cb194da

spec: add MIT to License field

This adds MIT License to licenses specified in spec's License field.

Change-Id: I40635546c35d11c777767740dea7bac2993bedf3

license: add MIT license

Some part of src/crash-stack/crash-stack-x86.c seems to be copied
from libunwind. The part has MIT license.

This adds MIT license text and an entry in the spec file.

Change-Id: I8796d9ad2e61cd0671470eaaae184c2d00e6615f

Release 6.0.4

This release brings:

 - Add stability_monitor user to the crash-service dbus policy

Change-Id: Ief93724cbf23771163186a35c1b69806bd32027f

Add stability_monitor user to the crash-service dbus policy

Change-Id: If47b06d92e09a566a195763012bb2cd19f31eb6e

dump_systemstate: Ensure extra-files and extra-programs are included by default

Needed as regular dump_systemstate functionality has been implemented
via .conf files.

Change-Id: I32d1c4cb954f6556c5782ff2e56f0f1c9d06acd4

dump_systemstate: Dump additional files and applications output

Change-Id: I2c1348a25f7da30c5a177489eb4e57aed5f730ea

dump_systemstate: Add flag to dump_systemstate to determine whether to run buxton2ctl or not

The next commit introduces a change that causes the crash-service not to
have to be run as root, so there is need for another way to determine if
dmesg or buxton2ctl should be run.

Change-Id: I56df8f4dd1b9d8a2f6f072c886e8fa26dda0a317

Release 6.0.3

This release brings:

 - Revert "Allow new special priv_livecoredump group members to use livedump API" commit

Change-Id: I10a2458fd1504f475b39e05a91bcd1edfb5a382f

Revert "Allow new special priv_livecoredump group members to use livedump API"

This reverts commit ab88e92b1eb4bae8d94523141afb2d59ffc5e6e6.

Change-Id: Ida972aa30aaa001ace379a5051ef654589119f76

Release 6.0.2

This release brings:

 - Run crash-service and crash-worker as crash_worker user
 - Fix the name of libcrash-service requirement in crash-worker_system-tests

Change-Id: Ib1e94342c475b20cde77bd10fbd21fd8a9ddfcb5

Change system_fw user to crash_worker

Change-Id: I79938533fc4475dfd4d9d95e5037fe29928ad0c7

Set supplementary groups for the crash-manager process

Change-Id: I8283c0ca4605d6e882836e4a02079c65dd1ad30c

Drop privileges when started by kernel

Change-Id: I3fa61c9f7b89f3fe7b8c7a8bedd8e852bdccf404

Configure the crash-service so that it doesn't require root privileges

Change-Id: I6bddc446a31449b7419074f10f795b8ee6c61714

Change crash-worker-libcrash-service requirement to libcrash-service

Change-Id: I1fcc2886404daea67b6be86b6930bb567a0c3f32