Re: [jamboost] Re: W32_GETREG broken

From: PAL (b84037_at_[hidden])
Date: 2004-02-11 18:56:49

> David Abrahams <dave_at_[hidden]> writes:
>
> > I can't figure out how to use it, but it has to be busted, since this
> > causes a crash:
> >
> > C:\Documents and Settings\dave>bjam -f-
> > ECHO [ W32_GETREG "HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows
NT\CurrentVersion" : "CurrentVersion" ] ;
> > ^Z
>
> OK, now rewritten and fixed. It had quite a few bugs.
> It still isn't complete; it doesn't handle a few of the registry
> value types mentioned here: http://tinyurl.com/2a5s3, but at least it
> doesn't crash.
>
Hmm... I have compared the original code with yours.
I found that
----------------------------------------------------------------------------
---
LPSTR szRootKey = newstr( lol_get(frame->args, 0)->string );
szKey = strchr(szRootKey , '\\');
*szKey = '\0'; ++szKey;
----------------------------------------------------------------------------
---
when " *szKey = '\0' " would raise an access violation.
I don't know why... I remember I have already test it and been sure that it
works.
But now I can't pass the test, either.
Could anybody show me what happened? Thanks.
-- Paul