[OpenBIOS] [PATCH 2/2] Don't map the page 0 to make NULL-dereferencing more obvious

Mark Cave-Ayland mark.cave-ayland at ilande.co.uk
Wed Apr 10 01:08:01 CEST 2013


On 09/04/13 20:49, Artyom Tarasenko wrote:

>> I've just committed a patch to fix this (on a second attempt as I
>> accidentally caught the NULL dereference patch in there too) so please feel
>> free to test.
>
> But we still don't need ihandle>phandle in (.property-by-name), do we?

Yes, because the properties in /chosen are ihandles (see page 23 of the 
IEEE1275 specification). So if you are still seeing strange behaviour 
that could be another bug - if you can post a test case, I'll take a look.


ATB,

Mark.



More information about the OpenBIOS mailing list