mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Kevin Klues <>
Subject Re: Review Request 49658: Updated elfio-3.1.patch to fix alignment bug properly.
Date Tue, 05 Jul 2016 21:25:41 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated July 5, 2016, 9:25 p.m.)

Review request for mesos and Benjamin Mahler.


Added bug and testing section

Bugs: MESOS-5767

Repository: mesos


The previous patch incorrectly fixed the alignment of an ELF note to 4
bytes (even for 64 bit ELF files). This happened to be OK for our
specific case of parsing '.note.ABI-tag' sections. However, the
alignment for notes in a 64 bit ELF section should be 8-byte aligned
(as the original code attempts to do). The problem with the original
code, however, is that it performs the alignment only on the name
string inside the note, rather than the entire note object itself.
This commit fixes this alignment issue.


  3rdparty/elfio-3.1.patch f8c20e92a0ec12e8a2f1d8e8ec54eac78ced707d 


Testing (updated)

make check


Kevin Klues

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message