Version 0.9.4

This release includes unconditionally scanning at start up (previous
version only scanned after an inotify or the long timer firstrang, when not
in test mode), a core based timestamp in the report, and unconditionally
setting the core_folder and processed_folder permissions to what we expect
them to be during start up validation.

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

Unconditionally scan at start

Originally there was an unconditional scan on start up followed by an
exit if in test mode.  With the changes I started for making things
more event driven, I moved that initial scan to an 'if test mode, scan
then exit' block.  We do actually want to always scan on start to get
existing reports processed promptly.

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

Always chmod our directories.

I observed the the modes of corewatcher's directories had changed on one
of my systems.  I don't know what would have caused that, but it led to
cores not being recorded in the filesystem.  To be robust we create the
directories if for some reason they do not exist at start up.  Similarly,
try to always insure they have the correct permissions at start up.

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

Add timestamp from core

The server has been attributing a timestamp to a crash report based on the
time the report is received.  This may differ greatly from the actual time
of the issue.  This is easily fixed with a stat of the corefile.

Unfortunately, the same can't as easily be done for getting the os-build,
which may also be incorrectly attributed on the client side based on the
/etc/os-release file's contents at the time of a crash's processing by
corewatcher, as opposed to at the actual time of the crash.

Tracking the combination of the reported os_build, core time, and receiving
time should give us more rich reporting and ability to discern problems
between different classes of users (eg: more/less network connectedness,
more/less aggressive adoption of updates).

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

Add inotify support, remove pthreads dep, bump version

Arjan noted a lot of activity on wake/poll.  This was when the server
was down and cores/reports were present to submit so there was work
to do.  But a lot of effort was generated to no gain.  The client should
be smarter with respect to sending reports on demand and minimizing
resource usage.  We still need additional state, parallelization and event
connectors to allow different threads to prepare reports for submission,
improve current report quality if high speed network (and thus debuginfo)
becomes available, do just report submission if low quality network
becomes available, etc.

This commit adds inotify support so we immediately attempt to process
cores and reduces the poll loop time to 15 minutes.  Lacking more
sophisticated network event driven logic, a slow poll loop can still
lead to reports eventually being submitted if they weren't able to be
fully submitted at inotify time.

In support of this a few functions are split out and GThreads/GMutexes
used instead of pthread.

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

Update example yaml after gdb update

William hacked gdb to list the path to the binary from which the symbols
came in addition to the file name and line number.

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

Remove extra unlink

This was the unlink of the actually processed file.  The primary duplicate
detection is in move_core().  Due to the two phases of processing in
scan_corefolder() it is a fully normal path for a core to get added
to the work queue and then a second add attempt likely happens at
reprocess_corefile().

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

Add some tentative duplicate handling

The corewatcher needs to be ratelimited a bit for extremely crashy
applications.  For example mate's marco will respawn as much as a hundred
times in a second.  In the case of two cores within the same second the
existing code collides in the hash table, so simply note that and unlink
the core file which is under processing currently instead of erroring
back up the stack to the main loop.  Do similarly for core files which
match within two characters at the end of the filename (ie: same time
mod 100 seconds).

This does have a bug though as the sent core is also being unlinked instead
of persisting, but it's a crude start anyway.  Still needs refining and
also the space needs tmp watched.

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

Fix apparent possible memory overrun in processing_queue

The queue is a simple array of strings of size defined by:
  #define MAX_PROCESSING_OOPS 10
but the add_to / remove_from functions were hard coded to wrap the tail and
head array indices at 100.

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

Version bump and change name in autotools

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

Update version number

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

Add systemd unit file

This should be a correct way to add a systemd unit file in autotools.  This
unit file simply allows systemd to run the corewatcher daemon.

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

Add COPYING file to Makefile.am

The COPYING file should be included in "the package".

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

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