Remove valgrind suppressions

We should either see the errors or fix them, imho.

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Remove unused/unneeded corewatcher-config bits

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Require allow-submit=yes and allow-pass-on=yes

Refuse to run if allow-submit=yes and allow-pass-on=yes are not set in the
config file.  These will be set by a prompt at first boot.

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Cope with oddities around umask and permission bits

For some reason the mask passed to mkdir() isn't (always?) honored, but the
directory starts with safe enough perm's and I can subsequently add the
ones I'd just asked for and not been given.

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Remove unused build_release

This is used in one place and simply hard coded there to be /etc/os-release
which is the new/current standard everywhere.

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Change from /tmp to /var/lib/corewatcher

We want dumps to be persistent across reboots, so tmpfs isn't the best
place for them.  Plus /tmp is a wild west of writability.  We'll allow full
writing of /var/lib/corewatcher, but not /var/lib/corewatcher/processed.
Still need a tmp watcher on this area to prune files.

Also this directory is an editorial choice by the distro maker, not a user
configurable option...remove user configurability.

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Remove release-info option

This is distribution creator editorial choice, not a user choice.

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Remove unlink option

This will be handled for us by tmpwatch

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Remove system config files

In Nitra these are owned by a system configuration files "package".

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Open /var/lib/corewatcher correctly after creation

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Shift files from /tmp to /var/lib/corewatcher

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Remove two old meego-isms

The /etc/os-release file is now standard and we have our own default server
for now and it isn't the meego one.

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Search for core_* instead of core.*

This is pretty arbitrary, but that happens to be what's checked into
nitra's config files currently.  Eventually we'll be called dynamically
by systemd probably when it receives things regarding which we've asked
to be notified.

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Fix autotools paths

I don't fully understand how/where autotools gets its default values, but
these don't seem to be set correctly if I don't explicitly set them.

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

A little more cleaning pre-production

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

A little more cleaning

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Correct memory management around os-release/version

Valgrind reminded me I should correctly dup the substring and free the
original full string.

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Add an example yaml file

This is an example of a low quality (no debuginfo present) core that was
processed/submitted.

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Clean out meego/tizen specific things and quiet compile warnings

Updated code to match nitra needs in terms of paths, config, info sent to
server.  Also removed a lot of code unused as a result of the code
updating, which was causing compile warnings due to our pedantic nitra
settings.

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

Fix config file path

For me 'make prefix=/usr libdir=/usr/lib64 sysconfdir=/etc install' put the
configuration files in /etc/corewatcher/, but these files weren't looking
in that subdirectory.

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

De-brand config file

Signed-off-by: Tim Pepper <timothy.c.pepper@linux.intel.com>

testmode wasn't getting set when it should have been, fix

Signed-off-by: William Douglas <william.douglas@intel.com>

remove incorrect comments

Signed-off-by: William Douglas <william.douglas@intel.com>

Remove useless check.

Since the crash submission/opt in was removed this check was changed
to only testmode, making test mode useless. Since we always send we
need to modify what testmode does now.

Signed-off-by: William Douglas <william.douglas@intel.com>

add short option check for test mode

Signed-off-by: William Douglas <william.douglas@intel.com>

fix compiler failure do to taking out crash submission code

Signed-off-by: William Douglas <william.douglas@intel.com>

add test that generates a core for corewatcher

Signed-off-by: William Douglas <william.douglas@intel.com>

Remove options to not submit crash info

Signed-off-by: William Douglas <william.douglas@intel.com>

Use CFLAGS for the configuration file as well

Signed-off-by: William Douglas <william.douglas@intel.com>

update gitignore for autoconf

Signed-off-by: William Douglas <william.douglas@intel.com>

Remove opt out check.

Always submit so remove check for opt out. Still need to remove the
configuration bits for this.

Signed-off-by: William Douglas <william.douglas@intel.com>

Update CFLAGS from old Makefile

Signed-off-by: William Douglas <william.douglas@intel.com>

Remove dbus interface to UI

Signed-off-by: William Douglas <william.douglas@intel.com>

Add check for malloc failing

Signed-off-by: William Douglas <william.douglas@intel.com>

Fix double free possibility

Signed-off-by: William Douglas <william.douglas@intel.com>

Update ioprio defines

We don't use RT priority so remove that.  Also 7 seemed like a
magic number so try and explain it and reference the kernel
docs where everything came from.
Signed-off-by: William Douglas <william.douglas@intel.com>

Take out Debian Etch work around.

Etch is old, move on.
Signed-off-by: William Douglas <william.douglas@intel.com>

Rename coredump scan function.

Doing this rename for clarity since scan_dmesg doesn't scan
dmesg and the kernel oops code actually does so it will take
that name.
Signed-off-by: William Douglas <william.douglas@intel.com>

Reorganize header, move objects under the appropriate source file

Part of kernel oops integration to corewatcher.
Signed-off-by: William Douglas <william.douglas@intel.com>

Update corewatcher to use autotools.

This involves major source restructuring and in the process
some files have been renamed as well as moved.  This was
done to ease the use of autotooling the package though
it shouldn't impair clarity of purpose for the files.

At this point there are further opportunities to improve
the packaging such as introducing #defines for some
file locations in the code.

Signed-off-by: William Douglas <william.douglas@intel.com>

Remove old define that was duplicating asm/unistd.h information.

This was for __NR_ioprio_set which is a syscall without a glibc
interface.  Since the define is already done in asm/unistd.h
for our architectures we no longer need to have it in corewatcher.
Signed-off-by: William Douglas <william.douglas@intel.com>

initial removal of corewatcher-applet

Signed-off-by: William Douglas <william.douglas@intel.com>

Rework corewatcher, clean valgrind warnings.

Major rework of corewatcher.  Corewatcher is now multithreaded
and will spawn a thread to process each corefile and will now
send a crash notification as soon it scans the /tmp/ folder.

There should be no more than one thread running gdb at a time
so we have a global lock to prevent that.

Corewatcher was also cleaned up to have 0 valgrind errors
(that are not suppressed, we have added a valgrind suppression
file for that purpose).

From here on any changes to corewatcher which cause a valgrind
error need to be discussed before being commited and ack'd
by a maintainer (and have its error added to the suppression
file).  Right now all suppressions are either glibc not doing
things that are not ansi c compliant or are in a library
corewatcher depends on (likely also running into glibc compliance
issues but I was unable to verify for all of the errors).

To test corewatcher for valgrind errors it should be run in
with no internet connection and with an internet connection
with multiple core files available to scan.

The following command should be used to test corewatcher with
valgrind (run from the source build directory):
G_SLICE=always-malloc G_DEBUG=gc-friendly valgrind \
 --partial-loads-ok=yes --read-var-info=yes --leak-check=full \
 --track-origins=yes --malloc-fill=AD --free-fill=DA \
 --suppressions=./corewatcher.supp --gen-suppressions=yes \
 ./corewatcher -d -n

Signed-off-by: William Douglas <william.douglas@intel.com>

corewatcher v0.26

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

Check getline return properly

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

corewatcher v0.25

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

clean up memory leaks

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

add meaningful package list details for files in coredump

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

corewatcher v0.24

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

skip over crashes that could cause looping

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

Revert "Remove rpm information gathering as we don't use it"

This reverts commit 6b98180a0081b109a973f609669256c7e2237122.

corewatcher 0.23

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

Remove rpm information gathering as we don't use it

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

corewatcher 0.22

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

Change behavior to avoid rerunning gdb each time we try scan corefiles

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

corewatcher 0.21

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

fix many temporary core detail files from flooding the system

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

corewatcher 0.20

corewatcher: libproxy support

add libproxy support for automatic proxy selection

Signed-off-by: Chris Leech <christopher.leech@linux.intel.com>

corewatcher: add private field to crash report

Private data will not be visible on the public web interface.

Signed-off-by: Chris Leech <christopher.leech@linux.intel.com>

corewatcher-config: add command line tool for editing the config file

Simple tool using the glib keyfile support to get and set key/value pairs.
Intended to be used to script configuration changes, such as.

# corewatcher-config --set allow-submit=yes

Default config file changes add a required group header for glib and whitespace
changes to reflect what happens when the glib keyfile routines read and then
write back the file.

Signed-off-by: Chris Leech <christopher.leech@linux.intel.com>

Version 0.19

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

Add new translations: es_MX, eu, uk, vi

Signed-off-by: Patrick McCarty <patrick.mccarty@intel.com>

Pull from Transifex: update existing translations

Signed-off-by: Patrick McCarty <patrick.mccarty@intel.com>

clean up some memory leaks

Send message to dbus once core is processed and once core is submitted

Debugged and tested keeping files to send until online

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

first stab at resending coredumps that were unsent

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

Version: 0.18

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

rework finding qml-launcher apps

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

add application name and crash id to the dbus sent signal

Signed-off-by: Chris Leech <christopher.leech@linux.intel.com>

pass oops struct instead of just a text blob, just to keep application name useable

Signed-off-by: Chris Leech <christopher.leech@linux.intel.com>

sent dbus signal and log to syslog on each oops, not just once per batch

Signed-off-by: Chris Leech <christopher.leech@linux.intel.com>

Version v0.17

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

update gdb command to output the core file

Finalize version v0.16

Signed-off-by: William Douglas <william.douglas@linux.intel.com>

Add the build release information from MeeGo to corewatcher

Fixup pointers that aren't initialized to NULL

Fixed crash scenario, redid wrapper scanning

Version 0.15

Allow corewatcher to try multiple submit urls

Bugfixes:
- Fixed issue with set_wrapped_app not getting correct executable.
- Various memory leaks plugged.

Enhancements:
- Read multiple urls from config, try until one works.
- Write url submitted to syslog
- Write coredump text to file

Have the process reported be the actual process run by meego-tablet-wrapper

oops need to an actual pid for the running process

Update default config to keep core dumps after processing

Keep the core generation origin for export

Add nice and IO nice to avoid grinding the system to a halt on a coredump

Unlink core files we can't find

disclaimer.

Adding example crash application.

Merge branch 'master' of gitorious.org:meego-developer-tools/corewatcher

No need to print this out to the console every start.

Also retain the text-version of the backtrace when config file requests.

l10n: Updated Walloon (wa) translation to 100%

New status: 13 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: First Walloon translation

New status: 5 messages complete with 0 fuzzies and 8 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Updated Chinese (Taiwan) (zh_TW) translation to 100%

New status: 13 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Added Portuguese locale

New status: 13 messages complete with 0 fuzzies and 0 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Arabic translation

New status: 5 messages complete with 0 fuzzies and 8 untranslated.

Transmitted-via: Transifex (www.transifex.net).

l10n: Persian trans

New status: 10 messages complete with 0 fuzzies and 3 untranslated.

Transmitted-via: Transifex (www.transifex.net).

Rev to 0.13 - fix locale install

Rev to 0.12