This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH v2] [BFD] Add support for reading msdos (MZ) executables.


On 01/03/2018 04:38 AM, Nick Clifton wrote:
> Hi Zebediah,
> 
>   Looking good - thanks for the updates.  Please remember however that
>   we cannot accept the patch until you have a copyright assignment in
>   place.  I can provide you with the details of how to request one if
>   you want.
> 
>   Also...
> 

Hmm, I'd appreciate that; the instructions on gnu.org are not readily
findable.

>> +static const bfd_target *
>> +msdos_object_p (bfd *abfd)
>> +{
>> +  struct external_DOS_hdr hdr;
>> +  bfd_byte buffer[2];
>> +  asection *section;
>> +  unsigned int size;
>> +
>> +  if (bfd_seek (abfd, (file_ptr) 0, SEEK_SET) != 0
>> +      || bfd_bread (&hdr, (bfd_size_type) sizeof (hdr), abfd) != sizeof(hdr))
> 
>   Isn't the external_DOS_hdr structure in this version of the patch larger 
>   than the version that was in your original submission ?  (Because of the 
>   inclusion of the dos_message field).  If so, does this mean that the code
>   might reject valid MZ binaries because they are too small ?
> 
> Cheers
>   Nick
> 
> 

Hmm, that's a good point. I'll send a revised patch.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]