platform/core/system/crash-worker.git
4 years agoFix 'full_core' system test 92/234992/1
Mateusz Moscicki [Tue, 26 May 2020 12:59:31 +0000 (14:59 +0200)]
Fix 'full_core' system test

Change-Id: I9bd5831f134141546fe435b9042b87a13ed39c1b

4 years agoAdd option to release the crashed process as soon as possible 96/234096/6
Mateusz Moscicki [Thu, 21 May 2020 09:28:12 +0000 (11:28 +0200)]
Add option to release the crashed process as soon as possible

Change-Id: Idd951c4b8c50ba1e29524339358475acdbacd382

4 years agoMerge "Read the /proc/<PID>/comm earlier" into tizen
Karol Lewandowski [Fri, 22 May 2020 14:12:03 +0000 (14:12 +0000)]
Merge "Read the /proc/<PID>/comm earlier" into tizen

4 years agoRead the /proc/<PID>/comm earlier 95/234095/2
Mateusz Moscicki [Thu, 21 May 2020 10:55:24 +0000 (12:55 +0200)]
Read the /proc/<PID>/comm earlier

Change-Id: Ied692bc4e770c9aa3b8f03f7a2515214bc40cb63

4 years agoClarify logs related to coredump handling 77/234077/2
Karol Lewandowski [Thu, 21 May 2020 10:48:30 +0000 (12:48 +0200)]
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

4 years agoRelease 6.0.17 91/232591/1 accepted/tizen/unified/20200510.220332 submit/tizen/20200506.154637 submit/tizen/20200507.084828
Mateusz Moscicki [Wed, 6 May 2020 15:11:51 +0000 (17:11 +0200)]
Release 6.0.17

This release brings fixes

Change-Id: Iebcb0ec695edf3c589507c9d9a667fac5bb8cab7

4 years agoFix: app ids repeats in app's manifest xmls search logic 86/232586/7
Wiktor Gerstenstein [Wed, 6 May 2020 13:09:27 +0000 (15:09 +0200)]
Fix: app ids repeats in app's manifest xmls search logic

Change-Id: I3fe0ca9cfef80077c9d3abb40fb439fc0bb87203

4 years agoFix Coverity issue 87/232587/1
Mateusz Moscicki [Wed, 6 May 2020 10:52:36 +0000 (12:52 +0200)]
Fix Coverity issue

dirname() could return pointer to the local variable

Change-Id: I886df1d2abf73d49f20c11e7ed5eef4595d1c6c5

4 years agoCheck if the file operation failed by lack of disk space 63/232463/3
Mateusz Moscicki [Tue, 5 May 2020 15:41:39 +0000 (17:41 +0200)]
Check if the file operation failed by lack of disk space

Change-Id: I3921e4631d9d2a84110af0681777bae15e62cfbc

4 years agoMerge "Returns unique file paths from the maps file" into tizen
Karol Lewandowski [Tue, 5 May 2020 19:36:57 +0000 (19:36 +0000)]
Merge "Returns unique file paths from the maps file" into tizen

4 years agoReturns unique file paths from the maps file 50/232450/1
Mateusz Moscicki [Tue, 5 May 2020 08:01:23 +0000 (10:01 +0200)]
Returns unique file paths from the maps file

This prevents duplicate entries in the *.so_info file

Change-Id: I49c4d4949d4d0075b056891557fe027d82b017a0

4 years agoRelease 6.0.16 17/232317/1 submit/tizen/20200430.145229
Mateusz Moscicki [Thu, 30 Apr 2020 13:17:48 +0000 (15:17 +0200)]
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

4 years agoAdd app's manifest xmls search logic 50/232150/10
Wiktor Gerstenstein [Tue, 28 Apr 2020 16:55:49 +0000 (18:55 +0200)]
Add app's manifest xmls search logic

Change-Id: I585cdc6e395efda52390c6059989d4da6afe5edb

4 years agoInclude packages in so_info file that are associated with *.ni.dll 61/231561/3
Mateusz Moscicki [Wed, 22 Apr 2020 10:04:25 +0000 (12:04 +0200)]
Include packages in so_info file that are associated with *.ni.dll

Change-Id: I114a866da9fc70915b616b39869d23cb9e7f22c7

4 years agoRemove duplicate g_object_unref 13/231513/2
INSUN PYO [Wed, 22 Apr 2020 07:39:39 +0000 (16:39 +0900)]
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

4 years agoRelease 6.0.15 03/231003/1 accepted/tizen/unified/20200419.223914 submit/tizen/20200416.123008
Karol Lewandowski [Thu, 16 Apr 2020 11:36:07 +0000 (13:36 +0200)]
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

4 years agoUse bool instead of int in remove.*() functions 38/230838/3
Mateusz Moscicki [Tue, 14 Apr 2020 11:36:19 +0000 (13:36 +0200)]
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

4 years agoMove unlocking and removing of the temp dir to a separate function 37/230837/2
Mateusz Moscicki [Tue, 14 Apr 2020 09:55:41 +0000 (11:55 +0200)]
Move unlocking and removing of the temp dir to a separate function

Change-Id: If38fcfda5737e8d39cb496692ca663e2a3d7c6a2

4 years agoAdd locking and cleaning temporary directory 63/230063/5
Mateusz Moscicki [Fri, 3 Apr 2020 13:42:39 +0000 (15:42 +0200)]
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

4 years agoFix report_basic test 65/230065/5
Mateusz Moscicki [Tue, 7 Apr 2020 08:30:24 +0000 (10:30 +0200)]
Fix report_basic test

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

Change-Id: I26fd47126d1d8b14ed8da6547f88275e2cbba116

4 years agoFix libcrash-service test 64/230064/5
Mateusz Moscicki [Tue, 7 Apr 2020 08:06:06 +0000 (10:06 +0200)]
Fix libcrash-service test

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

Change-Id: I71c6f2c401c293769caa816fe2e767b30578d7b6

4 years agocrash-manager: Clarify when tizen-manifest.xml is copied 18/230218/2
Karol Lewandowski [Wed, 8 Apr 2020 14:33:50 +0000 (16:33 +0200)]
crash-manager: Clarify when tizen-manifest.xml is copied

Change-Id: Iba48fc54f9809c077b5ebc0fbe8463186abd767e

4 years agocrash-manager: Fix free of invalid pointer returned by dirname() 17/230217/2
Karol Lewandowski [Wed, 8 Apr 2020 14:33:18 +0000 (16:33 +0200)]
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

4 years agoMerge "Add tizen-manifest.xml to crash-reports" into tizen
Karol Lewandowski [Wed, 8 Apr 2020 13:02:40 +0000 (13:02 +0000)]
Merge "Add tizen-manifest.xml to crash-reports" into tizen

4 years agoFix build id reading when the binary don't have it 56/230056/1
Mateusz Moscicki [Tue, 7 Apr 2020 11:27:12 +0000 (13:27 +0200)]
Fix build id reading when the binary don't have it

Change-Id: I82db306cad980104e096e8b738db24fc5e3547bb

4 years agoAdd tizen-manifest.xml to crash-reports 03/225403/23
Wiktor Gerstenstein [Wed, 19 Feb 2020 16:37:21 +0000 (17:37 +0100)]
Add tizen-manifest.xml to crash-reports

Change-Id: I6fc7f723ad33cd825e2c5f85b42c7b134a0f1f8b

4 years agoRelease 6.0.14 48/229648/3 accepted/tizen/unified/20200403.153141 submit/tizen/20200402.130550
Karol Lewandowski [Thu, 2 Apr 2020 11:40:03 +0000 (13:40 +0200)]
Release 6.0.14

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

Change-Id: I1aada1d58681272703b7c64181d0c7aceccc3d84

4 years agodump_systemstate: report ran command faithfully 52/229652/2
Michal Bloch [Thu, 2 Apr 2020 12:14:45 +0000 (14:14 +0200)]
dump_systemstate: report ran command faithfully

Change-Id: Id4b830804ae420b60a7250c220a56a5695f80cff
Signed-off-by: Michal Bloch <m.bloch@samsung.com>
4 years agoMerge "dump_systemstate: Add /opt/usr disk usage section" into tizen
Karol Lewandowski [Thu, 2 Apr 2020 13:02:37 +0000 (13:02 +0000)]
Merge "dump_systemstate: Add /opt/usr disk usage section" into tizen

4 years agoMerge "dump_systemstate: Use compile-time parameter check in get_disk_used_percent...
Karol Lewandowski [Thu, 2 Apr 2020 12:59:48 +0000 (12:59 +0000)]
Merge "dump_systemstate: Use compile-time parameter check in get_disk_used_percent()" into tizen

4 years agoMerge "Fix cmp_backtraces test" into tizen
Karol Lewandowski [Thu, 2 Apr 2020 12:58:04 +0000 (12:58 +0000)]
Merge "Fix cmp_backtraces test" into tizen

4 years agoMerge "packaging: remove libebl dependency" into tizen
Karol Lewandowski [Thu, 2 Apr 2020 12:49:38 +0000 (12:49 +0000)]
Merge "packaging: remove libebl dependency" into tizen

4 years agoFix cmp_backtraces test 66/228966/2
Mateusz Moscicki [Thu, 26 Mar 2020 13:23:28 +0000 (14:23 +0100)]
Fix cmp_backtraces test

Change-Id: Ia628adfb80a4945d34e9847e864c903106dba3e6

4 years agodump_systemstate: Add /opt/usr disk usage section 47/229647/3
Karol Lewandowski [Thu, 2 Apr 2020 11:37:03 +0000 (13:37 +0200)]
dump_systemstate: Add /opt/usr disk usage section

Change-Id: Ib52cba397f7272c14dcb7623efc85535d4d98d8e

4 years agodump_systemstate: Use compile-time parameter check in get_disk_used_percent() 46/229646/3
Karol Lewandowski [Thu, 2 Apr 2020 11:32:36 +0000 (13:32 +0200)]
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

4 years agodump_systemstate: Also list files when reporting /tmp disk usage 45/229645/1
Karol Lewandowski [Thu, 2 Apr 2020 08:55:30 +0000 (10:55 +0200)]
dump_systemstate: Also list files when reporting /tmp disk usage

Change-Id: I0ea99ca0a05a7d620a4462d6dc44257aa6e4bee0

4 years agopackaging: remove libebl dependency 41/229241/1
Dongkyun Son [Mon, 30 Mar 2020 12:23:37 +0000 (21:23 +0900)]
packaging: remove libebl dependency

libebl is not needed anymore.

Change-Id: If88fa16b361c70c9155e5240a03cbede3e235074
Signed-off-by: Dongkyun Son <dongkyun.s@samsung.com>
4 years agoRelease 6.0.13 66/228366/1 accepted/tizen/unified/20200321.043827 submit/tizen/20200320.153607
Karol Lewandowski [Fri, 20 Mar 2020 15:12:50 +0000 (16:12 +0100)]
Release 6.0.13

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

Change-Id: I6bbc308e0d15b1872e43fd5c5f89142d76b9943d

4 years agosystem tests: Sort entries 04/228204/1
Karol Lewandowski [Thu, 19 Mar 2020 14:31:38 +0000 (15:31 +0100)]
system tests: Sort entries

Change-Id: I8b5fea6bcd1a9cc50b9efdf96820b444a96d534f

4 years agoAllow disabling so_info generation via config file 03/228203/1
Karol Lewandowski [Thu, 19 Mar 2020 14:06:45 +0000 (15:06 +0100)]
Allow disabling so_info generation via config file

Change-Id: I89d3b687225a7a85498344c9dd84c17ba4f3e33b

4 years agoRelease 6.0.12 53/227553/1 accepted/tizen/unified/20200315.214856 submit/tizen/20200313.115636
Karol Lewandowski [Thu, 12 Mar 2020 12:34:30 +0000 (13:34 +0100)]
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

4 years agodump_systemstate: Allow passing arbitrary flags for cmdflag= usage 87/227387/3
Karol Lewandowski [Wed, 11 Mar 2020 12:45:37 +0000 (13:45 +0100)]
dump_systemstate: Allow passing arbitrary flags for cmdflag= usage

Change-Id: I0099e5248f5665af41dbbc22ef17932213f68abd

4 years agoSave in the *.so_info records related to C# DLL files. 46/226546/9
Mateusz Moscicki [Tue, 3 Mar 2020 13:14:34 +0000 (14:14 +0100)]
Save in the *.so_info records related to C# DLL files.

Change-Id: Ibee68bec0fdf3ae07c64b1240093269ce51ec6a6

4 years agoCopy address of regions related to C# DLL files. 45/226545/9
Mateusz Moscicki [Tue, 3 Mar 2020 12:43:08 +0000 (13:43 +0100)]
Copy address of regions related to C# DLL files.

Change-Id: Ic2e7e4b78d0ba85efcb6c3735152c5a5bffb395e

4 years agoChange the way the process name is reading 35/227535/2
Mateusz Moscicki [Wed, 11 Mar 2020 09:38:38 +0000 (10:38 +0100)]
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>
4 years agoMerge "dump_systemstate: Move most of the runnable programs to configuration files...
Karol Lewandowski [Wed, 11 Mar 2020 13:00:39 +0000 (13:00 +0000)]
Merge "dump_systemstate: Move most of the runnable programs to configuration files" into tizen

4 years agodump_systemstate ordering tests 97/227097/1
Michal Bloch [Mon, 9 Mar 2020 15:24:09 +0000 (16:24 +0100)]
dump_systemstate ordering tests

Change-Id: I17a0dac49bb8b013462d76c8f78be2693c356b3c
Signed-off-by: Michal Bloch <m.bloch@samsung.com>
4 years agodump_systemstate: Move most of the runnable programs to configuration files 76/226276/11
Karol Lewandowski [Fri, 28 Feb 2020 13:50:25 +0000 (14:50 +0100)]
dump_systemstate: Move most of the runnable programs to configuration files

Change-Id: I2b68d40de0f4309cc3a25dd6b2786bf5e0ff0902

4 years agodump_systemstate config: add `order` config entry 67/227067/2
Michal Bloch [Fri, 6 Mar 2020 13:21:45 +0000 (14:21 +0100)]
dump_systemstate config: add `order` config entry

Sort thereby

Change-Id: I1bfe5484a50763297048924d98374b0eaee66cf6
Signed-off-by: Michal Bloch <m.bloch@samsung.com>
4 years agodump_systemstate: Allow extra programs to be turned on/off based on a cmdline flag 83/226283/5
Karol Lewandowski [Fri, 28 Feb 2020 10:25:01 +0000 (11:25 +0100)]
dump_systemstate: Allow extra programs to be turned on/off based on a cmdline flag

Change-Id: I287c46d5286e622c58d772b8063e62f66db82e51

4 years agoconfig: Ignore case of ReportType key values 09/226209/5
Karol Lewandowski [Thu, 27 Feb 2020 17:48:12 +0000 (18:48 +0100)]
config: Ignore case of ReportType key values

Change-Id: Ifb46e382eaf3b56681103b4220a85a93dd9e9847

4 years agoAdd informative messages for crash-worker processes running with high LogLevel 08/226208/6
Karol Lewandowski [Thu, 27 Feb 2020 17:04:16 +0000 (18:04 +0100)]
Add informative messages for crash-worker processes running with high LogLevel

Change-Id: Id7a6e66d3dd371b0efb961195daca98ef42b5260

4 years agolog: Drop _SE logs 07/226207/6
Karol Lewandowski [Wed, 26 Feb 2020 08:29:56 +0000 (09:29 +0100)]
log: Drop _SE logs

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

Change-Id: If33cd93240ddf6d30e07ada9f23342ae309fce4d

4 years agolog: Make LogLevel statically configurable 06/226206/6
Karol Lewandowski [Wed, 26 Feb 2020 08:24:40 +0000 (09:24 +0100)]
log: Make LogLevel statically configurable

Change-Id: I6d38d777ec5aaef0e8ba71d6d1c91907c4a68057

4 years agoMove system-tests to main .spec 14/225814/4
Karol Lewandowski [Mon, 24 Feb 2020 14:44:54 +0000 (15:44 +0100)]
Move system-tests to main .spec

Change-Id: I675555baab83ce3051401757054bde3cf4317271

4 years agoRelease 6.0.11: Hotfix: Do not return pointer to member of destroyed pkgmgr structure 44/225944/2 accepted/tizen/unified/20200227.121108 submit/tizen/20200225.153334
Karol Lewandowski [Tue, 25 Feb 2020 14:51:38 +0000 (15:51 +0100)]
Release 6.0.11: Hotfix: Do not return pointer to member of destroyed pkgmgr structure

Change-Id: Ibe121a4d98cec1fe4aaaf61dd8b321f0c88ddd9a

4 years agoRelease 6.0.10 47/225647/2 accepted/tizen/unified/20200225.224245 submit/tizen/20200221.151318
Karol Lewandowski [Fri, 21 Feb 2020 13:10:17 +0000 (14:10 +0100)]
Release 6.0.10

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

Change-Id: I49bf203e836bf610ebb8836adb9af45cb23cabbb

4 years agoIntroduce list of processes excluded from dumping 48/225548/4
Karol Lewandowski [Thu, 20 Feb 2020 15:45:07 +0000 (16:45 +0100)]
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

4 years agoutil: Refactor get_cmd_info 46/225646/2
Karol Lewandowski [Fri, 21 Feb 2020 10:54:24 +0000 (11:54 +0100)]
util: Refactor get_cmd_info

Change-Id: Ie7c0584487367f7c059c49b0cc309ef5ac7e6542

4 years agoJanitorial: '<foo>*' -> '<foo> *' 46/225546/3
Karol Lewandowski [Thu, 20 Feb 2020 18:26:08 +0000 (19:26 +0100)]
Janitorial: '<foo>*' -> '<foo> *'

Place pointer signature consistently with rest of codebase.

Change-Id: Ie183a11724fe886036cf2929ca6acdfac989da9b

4 years agoutil: Add function to normalize paths produced by kernel.core_pattern %E 45/225545/3
Karol Lewandowski [Thu, 20 Feb 2020 15:46:03 +0000 (16:46 +0100)]
util: Add function to normalize paths produced by kernel.core_pattern %E

Change-Id: I0afbb7d946d16b2fff293c8697ae2e6d7db53e43

4 years agoComplain when livecoredumper is needed and not installed 44/225544/2
Karol Lewandowski [Thu, 13 Feb 2020 08:30:43 +0000 (09:30 +0100)]
Complain when livecoredumper is needed and not installed

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

Change-Id: If5ddddee6b99784d7ada9a93e89d3ad975facb6b

4 years agoconfig: Be more verbose about consequence of ExtraScript OOM condition 47/225547/2
Karol Lewandowski [Thu, 20 Feb 2020 18:26:59 +0000 (19:26 +0100)]
config: Be more verbose about consequence of ExtraScript OOM condition

Change-Id: I7add0d5df5fc2a0369c011574eff7a1eccd8343f

4 years agoRelease 6.0.9 75/225275/1 accepted/tizen/unified/20200219.130207 submit/tizen/20200218.160617
Karol Lewandowski [Mon, 17 Feb 2020 10:02:07 +0000 (11:02 +0100)]
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

4 years agoMagic shall not prevail 45/225145/3
Michal Bloch [Mon, 17 Feb 2020 16:07:32 +0000 (17:07 +0100)]
Magic shall not prevail

Change-Id: I01eded05dc377efd992a2fa175891ac6765b4114
Signed-off-by: Michal Bloch <m.bloch@samsung.com>
4 years agoSimplify struct crash_info setup 16/225016/5
Karol Lewandowski [Fri, 14 Feb 2020 16:20:51 +0000 (17:20 +0100)]
Simplify struct crash_info setup

Change-Id: Ibbb13c4534b89d6e1ee4b951a438ae1748ae30d2

4 years agoUse package id as prefix for report files 82/224882/2
Karol Lewandowski [Wed, 12 Feb 2020 15:00:15 +0000 (16:00 +0100)]
Use package id as prefix for report files

Change-Id: I37ea45e2c0a262e9bcd2bb3bcdb97edba83cbc7d

4 years agocrash-manager: Be a bit more verbose about consequence of missing TID 80/224880/2
Karol Lewandowski [Thu, 13 Feb 2020 14:24:27 +0000 (15:24 +0100)]
crash-manager: Be a bit more verbose about consequence of missing TID

Change-Id: If5b5a4c5050c9fc5afea233311de6145222c445f

4 years agoDrop sys-assert leftovers 79/224879/1
Karol Lewandowski [Thu, 13 Feb 2020 08:27:41 +0000 (09:27 +0100)]
Drop sys-assert leftovers

Change-Id: Ia220cc73a1b1e60f165aedd58cdfe9360c8de63b

4 years agoMerge "Adjust codebase for iniparser upgrade (4.x)" into tizen accepted/tizen/unified/20200213.121625 submit/tizen/20200203.003549 submit/tizen/20200205.003954
Karol Lewandowski [Mon, 3 Feb 2020 11:51:21 +0000 (11:51 +0000)]
Merge "Adjust codebase for iniparser upgrade (4.x)" into tizen

4 years agoRelease 6.0.8 68/223168/1 accepted/tizen/unified/20200129.022638 submit/tizen/20200123.165339
Karol Lewandowski [Thu, 23 Jan 2020 16:49:04 +0000 (17:49 +0100)]
Release 6.0.8

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

Change-Id: I2bd9cc0a7979dd92e0fe819e0eeca0ec37afb28c

4 years agoFix livedump API DBus access policy 61/223161/2
Karol Lewandowski [Thu, 23 Jan 2020 12:43:46 +0000 (13:43 +0100)]
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

4 years agoAdd missing system test to spec 83/223083/1
Karol Lewandowski [Wed, 22 Jan 2020 14:51:26 +0000 (15:51 +0100)]
Add missing system test to spec

Change-Id: I7d7b4d2a91e40938e65cb9153ebfc8d0e320876e

4 years agoRelease 6.0.7 12/222912/3 submit/tizen/20200121.141527 submit/tizen/20200123.105228
Karol Lewandowski [Tue, 21 Jan 2020 09:54:05 +0000 (10:54 +0100)]
Release 6.0.7

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

Change-Id: I964c7f8a58ac96f3055403d9a5c33aee223747e3

4 years agoAdd system test for legacy signal 55/222955/2
Karol Lewandowski [Tue, 21 Jan 2020 11:25:41 +0000 (12:25 +0100)]
Add system test for legacy signal

Change-Id: If4e2d007ebac2de6a11d64137c9d93e5efc75b6d

4 years agoAdd option to support legacy DBus 'ProcessCrashed' signal 91/222891/3
Kunhoon Baik [Tue, 21 Jan 2020 07:47:59 +0000 (16:47 +0900)]
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

4 years agoAdjust codebase for iniparser upgrade (4.x) 79/222079/1
Karol Lewandowski [Thu, 9 Jan 2020 10:22:00 +0000 (11:22 +0100)]
Adjust codebase for iniparser upgrade (4.x)

Change-Id: I3d8e613b8d1cccbbbf84b7dceac5abde57578fe6

4 years agoRelease 6.0.6 72/221772/1 accepted/tizen/unified/20200108.131512 submit/tizen/20200107.101346
Karol Lewandowski [Tue, 7 Jan 2020 09:24:04 +0000 (10:24 +0100)]
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

4 years agocrash-service: Restrict allow livedump API via privilege 59/220359/6
Karol Lewandowski [Tue, 17 Dec 2019 13:58:15 +0000 (14:58 +0100)]
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

4 years agocrash-manager: Make permission handling in maps more abstract 23/221523/3
Karol Lewandowski [Fri, 3 Jan 2020 08:16:48 +0000 (09:16 +0100)]
crash-manager: Make permission handling in maps more abstract

This also fixes gcc9.2 build error.

Change-Id: Iae099a366063cd69cca840bd83121a8930d89088

4 years agoRelease 6.0.5 81/220481/3 submit/tizen/20191220.194729
Karol Lewandowski [Wed, 18 Dec 2019 14:36:27 +0000 (15:36 +0100)]
Release 6.0.5

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

Change-Id: Ic01f780a2bb3868e48f02bbe95ac725adcb384cd

4 years agoconfig: Ensure freed pointers are set to null 45/220745/1
Karol Lewandowski [Fri, 20 Dec 2019 19:32:46 +0000 (20:32 +0100)]
config: Ensure freed pointers are set to null

Change-Id: I779cff27a6873187e5f17415469864d6ccd0d1b7

4 years agoconfig: Search for configuration also under crash-manager.conf.d directory 73/220173/5
Karol Lewandowski [Wed, 18 Dec 2019 09:41:09 +0000 (10:41 +0100)]
config: Search for configuration also under crash-manager.conf.d directory

Change-Id: I9c5680eafb467a0514feec5225cb794609ba1823

4 years agoconfig: Separate config loading from processing 72/220172/5
Karol Lewandowski [Fri, 13 Dec 2019 12:11:08 +0000 (13:11 +0100)]
config: Separate config loading from processing

Change-Id: Ief38894e03904f9947ee72773f07044280f62b41

4 years agocrash-service: Stability monitor does not need to own crash-worker's dbus name 58/220358/1
Karol Lewandowski [Tue, 17 Dec 2019 13:56:06 +0000 (14:56 +0100)]
crash-service: Stability monitor does not need to own crash-worker's dbus name

Change-Id: Ida73ce27ac6ea040dcffb81f3a61e4096dc81940

4 years agoFix file permissions 71/220171/1
Karol Lewandowski [Fri, 13 Dec 2019 10:46:13 +0000 (11:46 +0100)]
Fix file permissions

Change-Id: I078c745cc56e1cf108997f2e76897dbe4cb194da

4 years agospec: add MIT to License field 05/218805/1 accepted/tizen/unified/20191208.221907 submit/tizen/20191128.084355 submit/tizen/20191128.095939 submit/tizen/20191129.160037 submit/tizen/20191202.125225 submit/tizen/20191206.142827
Adrian Szyndela [Thu, 28 Nov 2019 07:53:56 +0000 (08:53 +0100)]
spec: add MIT to License field

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

Change-Id: I40635546c35d11c777767740dea7bac2993bedf3

4 years agolicense: add MIT license 27/218727/1 submit/tizen/20191127.103525
Adrian Szyndela [Wed, 27 Nov 2019 10:16:36 +0000 (11:16 +0100)]
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

4 years agoRelease 6.0.4 43/218643/2 accepted/tizen/unified/20191128.001532 submit/tizen/20191126.105153
Mateusz Moscicki [Tue, 26 Nov 2019 09:03:02 +0000 (10:03 +0100)]
Release 6.0.4

This release brings:

 - Add stability_monitor user to the crash-service dbus policy

Change-Id: Ief93724cbf23771163186a35c1b69806bd32027f

4 years agoAdd stability_monitor user to the crash-service dbus policy 42/218642/2
Mateusz Moscicki [Tue, 26 Nov 2019 09:02:31 +0000 (10:02 +0100)]
Add stability_monitor user to the crash-service dbus policy

Change-Id: If47b06d92e09a566a195763012bb2cd19f31eb6e

4 years agodump_systemstate: Ensure extra-files and extra-programs are included by default 45/218245/3
Karol Lewandowski [Wed, 20 Nov 2019 15:57:08 +0000 (16:57 +0100)]
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

4 years agodump_systemstate: Dump additional files and applications output 31/217031/6
Mateusz Moscicki [Tue, 5 Nov 2019 13:15:51 +0000 (14:15 +0100)]
dump_systemstate: Dump additional files and applications output

Change-Id: I2c1348a25f7da30c5a177489eb4e57aed5f730ea

4 years agodump_systemstate: Add flag to dump_systemstate to determine whether to run buxton2ctl... 63/215763/8
Mateusz Moscicki [Mon, 14 Oct 2019 11:04:56 +0000 (13:04 +0200)]
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

4 years agoRelease 6.0.3 66/218466/1 submit/tizen/20191122.132437
Mateusz Moscicki [Fri, 22 Nov 2019 13:06:13 +0000 (14:06 +0100)]
Release 6.0.3

This release brings:

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

Change-Id: I10a2458fd1504f475b39e05a91bcd1edfb5a382f

4 years agoRevert "Allow new special priv_livecoredump group members to use livedump API" 65/218465/1
Mateusz Moscicki [Fri, 22 Nov 2019 13:04:35 +0000 (14:04 +0100)]
Revert "Allow new special priv_livecoredump group members to use livedump API"

This reverts commit ab88e92b1eb4bae8d94523141afb2d59ffc5e6e6.

Change-Id: Ida972aa30aaa001ace379a5051ef654589119f76

4 years agoRelease 6.0.2 61/218361/2 submit/tizen/20191121.124127
Mateusz Moscicki [Thu, 21 Nov 2019 11:15:47 +0000 (12:15 +0100)]
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

4 years agoChange system_fw user to crash_worker 60/218360/2
Mateusz Moscicki [Wed, 20 Nov 2019 11:03:43 +0000 (12:03 +0100)]
Change system_fw user to crash_worker

Change-Id: I79938533fc4475dfd4d9d95e5037fe29928ad0c7

4 years agoSet supplementary groups for the crash-manager process 08/218008/3
Mateusz Moscicki [Mon, 18 Nov 2019 08:24:40 +0000 (09:24 +0100)]
Set supplementary groups for the crash-manager process

Change-Id: I8283c0ca4605d6e882836e4a02079c65dd1ad30c

4 years agoDrop privileges when started by kernel 52/217352/4
Mateusz Moscicki [Thu, 7 Nov 2019 10:41:59 +0000 (11:41 +0100)]
Drop privileges when started by kernel

Change-Id: I3fa61c9f7b89f3fe7b8c7a8bedd8e852bdccf404

4 years agoConfigure the crash-service so that it doesn't require root privileges 64/215764/13
Mateusz Moscicki [Mon, 14 Oct 2019 12:00:40 +0000 (14:00 +0200)]
Configure the crash-service so that it doesn't require root privileges

Change-Id: I6bddc446a31449b7419074f10f795b8ee6c61714