From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============8513853234410416108==" MIME-Version: 1.0 From: Mark Wielaard To: elfutils-devel@lists.fedorahosted.org Subject: Re: Fuzzing elfutils Date: Mon, 08 Dec 2014 10:14:21 +0100 Message-ID: <1418030061.18974.29.camel@bordewijk.wildebeest.org> In-Reply-To: 5484F850.8020605@mccme.ru --===============8513853234410416108== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable On Mon, 2014-12-08 at 04:01 +0300, Alexander Cherepanov wrote: > On 2014-12-04 19:03, Mark Wielaard wrote: > > Good news, the asserts from readelf-asserts.tar.gz don't trigger anymore > > and the command seems to run fine. > = > Sorry, I'm still seeing "readelf: readelf.c:7751: = > print_debug_exception_table: Assertion `readp =3D=3D action_table' failed= ." = > on 437b7cf1. Right, sorry, I didn't include -e in my tests. The assert is "right" the Action Table should immediately follow the Call site table. But we should just report invalid data if that isn't the case instead of asserting. Patch attached (and on the mjw/pending branch). Thanks, Mark --===============8513853234410416108== Content-Type: text/x-patch MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="0001-readelf-Report-invalid-data-if-action-table-doesn-t-.patch" RnJvbSBhY2ZhNDVjNDE3ZjdjNGZlYjM0ZDkxZDgxNTZmZDM5NGJmMGE1ZGY4IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQ0KRnJvbTogTWFyayBXaWVsYWFyZCA8bWp3QHJlZGhhdC5jb20+DQpEYXRl OiBNb24sIDggRGVjIDIwMTQgMTA6MTE6NDMgKzAxMDANClN1YmplY3Q6IFtQQVRDSF0gcmVhZGVs ZjogUmVwb3J0IGludmFsaWQgZGF0YSBpZiBhY3Rpb24gdGFibGUgZG9lc24ndCBmb2xsb3cNCiBj YWxsIHNpdGUgdGFibGUuDQoNCkRvbid0IGFzc2VydC4NCg0KU2lnbmVkLW9mZi1ieTogTWFyayBX aWVsYWFyZCA8bWp3QHJlZGhhdC5jb20+DQotLS0NCiBzcmMvQ2hhbmdlTG9nIHwgNSArKysrKw0K IHNyYy9yZWFkZWxmLmMgfCAzICsrLQ0KIDIgZmlsZXMgY2hhbmdlZCwgNyBpbnNlcnRpb25zKCsp LCAxIGRlbGV0aW9uKC0pDQoNCmRpZmYgLS1naXQgYS9zcmMvQ2hhbmdlTG9nIGIvc3JjL0NoYW5n ZUxvZw0KaW5kZXggMWMzY2YyZi4uNDc3MWRjOSAxMDA2NDQNCi0tLSBhL3NyYy9DaGFuZ2VMb2cN CisrKyBiL3NyYy9DaGFuZ2VMb2cNCkBAIC0xLDMgKzEsOCBAQA0KKzIwMTQtMTItMDggIE1hcmsg V2llbGFhcmQgIDxtandAcmVkaGF0LmNvbT4NCisNCisJKiByZWFkZWxmLmMgKHByaW50X2RlYnVn X2V4Y2VwdGlvbl90YWJsZSk6IFJlcG9ydCBpbnZhbGlkIGRhdGEgaWYNCisJYWN0aW9uIHRhYmxl IGRvZXNuJ3QgaW1tZWRpYXRlbHkgZm9sbG93IGNhbGwgc2l0ZSB0YWJsZS4NCisNCiAyMDE0LTEy LTA3ICBNYXJrIFdpZWxhYXJkICA8bWp3QHJlZGhhdC5jb20+DQogDQogCSogcmVhZGVsZi5jIChw cmludF9kZWJ1Z19saW5lX3NlY3Rpb24pOiBtYXhfb3BzX3Blcl9pbnN0ciBjYW5ub3QNCmRpZmYg LS1naXQgYS9zcmMvcmVhZGVsZi5jIGIvc3JjL3JlYWRlbGYuYw0KaW5kZXggZmE5ZWRlMS4uMWRi NTRjNiAxMDA2NDQNCi0tLSBhL3NyYy9yZWFkZWxmLmMNCisrKyBiL3NyYy9yZWFkZWxmLmMNCkBA IC03NzQ4LDcgKzc3NDgsOCBAQCBwcmludF9kZWJ1Z19leGNlcHRpb25fdGFibGUgKER3ZmxfTW9k dWxlICpkd2ZsbW9kIF9fYXR0cmlidXRlX18gKCh1bnVzZWQpKSwNCiAJCSAgICAgICAiICAgICAg ICBBY3Rpb246ICAgICAgICAgICAgJXVcbiIpLA0KIAkgICAgICB1KyssIGNhbGxfc2l0ZV9zdGFy dCwgY2FsbF9zaXRlX2xlbmd0aCwgbGFuZGluZ19wYWQsIGFjdGlvbik7DQogICAgIH0NCi0gIGFz c2VydCAocmVhZHAgPT0gYWN0aW9uX3RhYmxlKTsNCisgIGlmIChyZWFkcCAhPSBhY3Rpb25fdGFi bGUpDQorICAgIGdvdG8gaW52YWxpZF9kYXRhOw0KIA0KICAgdW5zaWduZWQgaW50IG1heF9hcl9m aWx0ZXIgPSAwOw0KICAgaWYgKG1heF9hY3Rpb24gPiAwKQ0KLS0gDQoxLjguMy4xDQoNCg== --===============8513853234410416108==--