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

[Tarl Neustaedter]

On 2013-Apr-8 15:00 , Artyom Tarasenko wrote:
> I guess ihandle>phandle shouldn't be called for the root entry. What
> do you think?

You're probably calling ihandle>phandle with a phandle. I would expect 
the return from active-package to be a phandle, not an ihandle.

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

[Artyom Tarasenko]

On Mon, Apr 8, 2013 at 9:15 PM, Tarl Neustaedter  wrote:
> On 2013-Apr-8 15:00 , Artyom Tarasenko wrote:
> > I guess ihandle>phandle shouldn't be called for the root entry. What
> > do you think?
>
> You're probably calling ihandle>phandle with a phandle. I would expect the
> return from active-package to be a phandle, not an ihandle.

You are absolutely right. Removing ihandle>phandle helped.
Thanks a lot!

linux/sparc and solaris/sparc under qemu blog:
http://tyom.blogspot.com/search/label/qemu