summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDenys Vlasenko <vda.linux@googlemail.com>2009-08-06 16:57:39 +0200
committerDenys Vlasenko <vda.linux@googlemail.com>2009-08-06 16:57:39 +0200
commit6c35b832998b807c35eef0ffc1cce93262d5550c (patch)
tree1ac2ea0df6e9acee5040e888fe928abe3b83328d
parentb64e1671afd1cd1c25604ffb3bf6f681a864b639 (diff)
downloadabrt-6c35b832998b807c35eef0ffc1cce93262d5550c.tar.gz
abrt-6c35b832998b807c35eef0ffc1cce93262d5550c.tar.xz
abrt-6c35b832998b807c35eef0ffc1cce93262d5550c.zip
fix vda's breakage: coredumps should be 644, not 600
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r--src/Hooks/CCpp.cpp6
1 files changed, 5 insertions, 1 deletions
diff --git a/src/Hooks/CCpp.cpp b/src/Hooks/CCpp.cpp
index b5b576b8..e821bb52 100644
--- a/src/Hooks/CCpp.cpp
+++ b/src/Hooks/CCpp.cpp
@@ -184,7 +184,11 @@ int main(int argc, char** argv)
snprintf(path + strlen(path), sizeof(path), "/%s", FILENAME_COREDUMP);
int fd;
- fd = open(path, O_WRONLY | O_CREAT | O_TRUNC, 0600);
+ /* We need coredumps to be readable by all, because
+ * process producing backtraces is run under the same UID
+ * as the crashed process.
+ * Thus 644, not 600 */
+ fd = open(path, O_WRONLY | O_CREAT | O_TRUNC, 0644);
if (fd < 0)
{
dd.Delete();