Re: [PATCH] bug fix for grub-pe2elf

2009-04-14 Thread Bean
Hi, ok, committed. On Tue, Apr 14, 2009 at 11:41 PM, Yoshinori K. Okuji wrote: > On Saturday 11 April 2009 21:07:51 Bean wrote: >> On Sat, Apr 11, 2009 at 6:24 PM, Yoshinori K. Okuji wrote: >> > On Saturday 11 April 2009 18:29:00 Bean wrote: >> >> Hi, >> >> >> >> When symbol name is 8 or less,

Re: [PATCH] bug fix for grub-pe2elf

2009-04-14 Thread Yoshinori K. Okuji
On Saturday 11 April 2009 21:07:51 Bean wrote: > On Sat, Apr 11, 2009 at 6:24 PM, Yoshinori K. Okuji wrote: > > On Saturday 11 April 2009 18:29:00 Bean wrote: > >> Hi, > >> > >> When symbol name is 8 or less, pe store it as short name, which is not > >> necessary null-terminated. > > > > Oh, I did

Re: [PATCH] bug fix for grub-pe2elf

2009-04-11 Thread Bean
On Sat, Apr 11, 2009 at 6:24 PM, Yoshinori K. Okuji wrote: > On Saturday 11 April 2009 18:29:00 Bean wrote: >> Hi, >> >> When symbol name is 8 or less, pe store it as short name, which is not >> necessary null-terminated. > > Oh, I didn't know that. Where is it documented? Hi, Recently, I found

Re: [PATCH] bug fix for grub-pe2elf

2009-04-11 Thread Yoshinori K. Okuji
On Saturday 11 April 2009 18:29:00 Bean wrote: > Hi, > > When symbol name is 8 or less, pe store it as short name, which is not > necessary null-terminated. Oh, I didn't know that. Where is it documented? Regards, Okuji ___ Grub-devel mailing list Gru

[PATCH] bug fix for grub-pe2elf

2009-04-11 Thread Bean
Hi, When symbol name is 8 or less, pe store it as short name, which is not necessary null-terminated. This patch take that into account and append a null character at the end. diff --git a/util/grub-pe2elf.c b/util/grub-pe2elf.c index d703d33..a2504d2 100644 --- a/util/grub-pe2elf.c +++ b/util/gr