On Wed, 2016-01-27 at 16:31 +0100, Ard Biesheuvel wrote:
On 27 January 2016 at 16:20, James Bottomley James.Bottomley@hansenpartnership.com wrote:
On Wed, 2016-01-27 at 13:17 +0100, Ard Biesheuvel wrote:
These patches have been in my personal queue for about two years, and have already been incorporated into the Ubuntu packaged version of sbsigntool.
Patch #1 and #2 fixes some generic issues, and patch #3 introduces the ARM and AArch64 magic values in the PE/COFF header check, which is all that is needed to get sbsign to sign ARM images.
Ard Biesheuvel (3): sbsigntool: remove doubly defined IMAGE_FILE_MACHINE_AMD64 sbsigntool: fix handling of zero sized sections sbsigntool: add support for ARM and Aarch64 PE/COFF images
src/coff/pe.h | 2 +- src/image.c | 48 +++++++++++--------- 2 files changed, 27 insertions(+), 23 deletions(-)
I can incorporate them into my build service package like I did for multi-sign. The problem is that Jeremy maintains the upstream and it's not moving:
git://kernel.ubuntu.com/jk/sbsigntool
I presume because Jeremy is now at Ozlabs? Do you have a different upstream URL?
To be honest, I wrote these 2+ years ago, so I don't remember exactly. Someone brought it to my attention that these are not upstream, but if that is still Jeremy instead of you, then yes, I suppose they need to be sent to Jeremy.
You mean
http://git.kernel.org/cgit/linux/kernel/git/jejb/sbsigntools.git/
It's where I keep my changes, but it's not where Ubuntu pulls from.
James