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

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