touch-packages team mailing list archive
-
touch-packages team
-
Mailing list archive
-
Message #43583
[Bug 1397340] Re: Integer overflow when processing giant field values
A likely candidate for the crash corresponding to this report is the
following:
https://errors.ubuntu.com/problem/2b929ca4aff09a8714851de0c45279b036386a10
** Tags added: trusty utopic vivid
** Changed in: whoopsie (Ubuntu)
Status: New => In Progress
** Changed in: whoopsie (Ubuntu)
Importance: Undecided => Medium
** Changed in: whoopsie (Ubuntu)
Assignee: (unassigned) => Brian Murray (brian-murray)
--
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to whoopsie in Ubuntu.
https://bugs.launchpad.net/bugs/1397340
Title:
Integer overflow when processing giant field values
Status in whoopsie package in Ubuntu:
In Progress
Bug description:
Ubuntu release: 12.04
Package version: 0.1.33
When parsing fields in a crash report file, whoopsie will reallocate
the value buffer when appending continuation lines. The current length
of the buffer is computed by pointer arithmetic and the result stored
in a signed integer. If the field value length reaches 2GB, then this
value will overflow, and become negative. This will then cause
whoopsie itself to abort, as it tries to allocate a huge amount of
memory.
I would expect whoopsie to cope with such large input (which may be
generated as the result of a memory-hungry process crashing and
creating a very large compressed+base64-encoded CoreDump).
By inspection, I see that this issue is still present in current
development versions: http://bazaar.launchpad.net/~ubuntu-
branches/ubuntu/vivid/whoopsie/vivid/view/head:/src/whoopsie.c#L402
I've attached a patch (created against the 0.1.33 sources, but should
apply with minimal issues against later versions), that resolves the
immediate issue. There's a more general question about the sanity of
loading the entire crash file into memory, too (particularly as the
CoreDump is never used unless the server requests it).
To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/whoopsie/+bug/1397340/+subscriptions
References