Solved

VBScript RegRead Function is not working as expected

Posted on 2011-02-17
13
1,406 Views
Last Modified: 2012-05-11
I have a simple function whereby I pass in the name of a Registry Key Path.
I have a const variable of HKLM = &H80000002.
I'm passing in the following Check on this path:
sKeyPath = "SYSTEM\CurrentControlSet\Services\Eventlog"

I just want to check to make sure that the "Eventlog" folder exists and that's it.
When I make the function call I've tried passing in the following different parameters:
If regKeyExists(sKeyPath) and If regKeyExists(HKLM & "\" & sKeyPath)

I'm getting this error and it's driving me nuts. I'm not sure what else it could be:
-2147024894

Please see code below.

Thanks Experts,
Wallace

Function regKeyExists(Key)
	On Error Resume Next
	
	Dim oShll, rKey
	Set oShll = CreateObject("WScript.Shell")
	rKey = oShll.RegRead(Key)
	MsgBox Err.Number
	If Err.Number <> 0 Then
		Err.Clear
		regKeyExists = False
	Else
		Err.Clear
		regKeyExists = True
	End If
	
	Set oShll = Nothing
End Function

Open in new window

0
Comment
Question by:wally_davis
  • 5
  • 5
  • 3
13 Comments
 
LVL 51

Accepted Solution

by:
Bill Prew earned 250 total points
ID: 34920242
You need a trailing backslash, as in:

regKeyExists("HKLM\SYSTEM\CurrentControlSet\Services\Eventlog\")

~bp
0
 
LVL 65

Expert Comment

by:RobSampson
ID: 34921284
Bill is right, but you can have your function make sure there is a trailing backslash as well.

Regards,

Rob.
Function regKeyExists(Key)
	On Error Resume Next
	
	If Right(Key, 1) <> "\" Then Key = Key & "\"
	Dim oShll, rKey
	Set oShll = CreateObject("WScript.Shell")
	rKey = oShll.RegRead(Key)
	MsgBox Err.Number
	If Err.Number <> 0 Then
		Err.Clear
		On Error GoTo 0
		regKeyExists = False
	Else
		Err.Clear
		On Error GoTo 0
		regKeyExists = True
	End If
	
	Set oShll = Nothing
End Function

Open in new window

0
 

Author Comment

by:wally_davis
ID: 34921954
Ok, I tried you're suggestions and to no avail. I'm doing some checking on the value that gets pass in to the Key variable (the parameter) in the Function regKeyExists(Key). I've tried prefixing the path sKeyPath = "SYSTEM\CurrentControlSet\Services\Eventlog\" with the the string of "HKLM" and the actual const variable where HKLM = &H80000002. So, it could look like one of the two prior to passing it in to the Function:
1. If regKeyExists(HKLM & "\" & sKeyPath & "\") Then OR
2. If regKeyExists("HKLM\" & sKeyPath & "\") Then

In the function itself I've displayed the value in the "Key" variable parameter to equate to this:
rKey = oShll.RegRead(Key)
So, var Key contains the Value "-2147483646\SYSTEM\CurrentControlSet\Services\Event Archiver Service" as it gets passed into this Key variable. After that, the rKey = Empty and the error I get is still -2147024894.
0
 
LVL 65

Expert Comment

by:RobSampson
ID: 34921997
Reg Read does not require any Hex constant values.  It only requires the standard user friendly string value of HKLM or HKCU, etc.

So, pass the value like this:

sKeyPath = "HKLM\SYSTEM\CurrentControlSet\Services\Event Archiver Service\"
If regKeyExists(sKeyPath) Then

Regards,

Rob.
sKeyPath = "HKLM\SYSTEM\CurrentControlSet\Services\Event Archiver Service\"

If regKeyExists(sKeyPath) Then
	MsgBox "Exists"
Else
	MsgBox "Does not exist"
End If

Function regKeyExists(Key)
	On Error Resume Next
	
	If Right(Key, 1) <> "\" Then Key = Key & "\"
	Dim oShll, rKey
	Set oShll = CreateObject("WScript.Shell")
	rKey = oShll.RegRead(Key)
	'MsgBox Err.Number
	If Err.Number <> 0 Then
		Err.Clear
		On Error GoTo 0
		regKeyExists = False
	Else
		Err.Clear
		On Error GoTo 0
		regKeyExists = True
	End If
	
	Set oShll = Nothing
End Function

Open in new window

0
 
LVL 51

Expert Comment

by:Bill Prew
ID: 34922591
As I mentioned in my first post, you need:

regKeyExists("HKLM\SYSTEM\CurrentControlSet\Services\Eventlog\")

~bp
0
 
LVL 65

Expert Comment

by:RobSampson
ID: 34922669
LOL! Yes, that's true.
0
Maximize Your Threat Intelligence Reporting

Reporting is one of the most important and least talked about aspects of a world-class threat intelligence program. Here’s how to do it right.

 

Author Comment

by:wally_davis
ID: 34927981
One last note guys, bp and Rob, you've both been a tremendous help.

I ran into one other snag. Let's say I have a KeyPath whose value is equal to:
HKLM\SYSTEM\CurrentControlSet\Services\Wallaces Key\".

Note the space in the last key "Wallaces Key". When I pass this in, it gives me an error of
-2147024894. I think this might have something to do with the fact the Key has a space (or could have multiple spaces). I tried passing the Key in with double quotes like this and it still fails when passed to the Function --> regKeyPath("""" & "HKLM\" & sKeyPath & "\" & """").

I'm sure there's a way through this problem. I just haven't found the fix for it. Thanks guys!
0
 

Author Comment

by:wally_davis
ID: 34928438
While I was thinking (ya, I know, dangerous huh) I'm also wondering if the Error Number of
-2147024894 is just another way of saying "This Subkey does not exist and this is the error I'm giving you as a result."

If you look at the function below, I get on of two values returned to the variable "rKey".
I get either a double-quote ("") and a Error Number of 0 if the Key exists or the var "rKey" will get "Empty" returned to it and an Error Number of -2147024894. Your experiences or thoughts on how you would handle it?
Function regKeyExists(Key)
	On Error Resume Next
	
	Dim oShll, rKey
	Set oShll = CreateObject("WScript.Shell")
	rKey = oShll.RegRead(Key)
	
	MsgBox Err.Number
	
	If Err.Number <> 0 Then
		Err.Clear
		On Error Goto 0
		regKeyExists = False
	Else
		Err.Clear
		On Error Goto 0
		regKeyExists = True		
	End If
End Function

Open in new window

0
 

Author Comment

by:wally_davis
ID: 34928477
Ah crap, I'm getting ahead of myself. I completely forgot that the key "HKLM\SYSTEM\CurrentControlSet\Services\Wallaces Key\" does exist and I'm still getting the Error Number of -2147024894 returned to me so I'm still trying to figure out why.
0
 
LVL 65

Assisted Solution

by:RobSampson
RobSampson earned 250 total points
ID: 34931104
Spaces do work.  I can read
sKeyPath = "HKLM\SYSTEM\CurrentControlSet\Services\.NET CLR Data\"

with no problem.

It must be a permissions thing.  Check permissions on that key to make sure the account you're running the script with has access to it.

Regards,

Rob.
0
 
LVL 65

Expert Comment

by:RobSampson
ID: 34931108
If you're on Windows Vista or 7, try running the script as Administrator.  You can do this by making a batch file with
wscript "C:\Temp\RegReadTest.vbs"

and then right click the batch, and Run As Administrator.

Rob.
0
 
LVL 51

Expert Comment

by:Bill Prew
ID: 34931160
Agreed, spaces are not a problem.

~bp
0
 

Author Closing Comment

by:wally_davis
ID: 35096269
Problem was two-fold. I needed a trailing backslash added to my code and it was also a permissions issue on the specific Server I was testing against. Thanks Bill and Rob!
0

Featured Post

Highfive + Dolby Voice = No More Audio Complaints!

Poor audio quality is one of the top reasons people don’t use video conferencing. Get the crispest, clearest audio powered by Dolby Voice in every meeting. Highfive and Dolby Voice deliver the best video conferencing and audio experience for every meeting and every room.

Join & Write a Comment

Well hello again!  Glad to see you've made it this far without giving up.  In this, the fourth installment of my popular series, I'm going to cover functions and subroutines, what they are, and why they are useful.  Just in case you stumbled onto th…
If you need to start windows update installation remotely or as a scheduled task you will find this very helpful.
Sending a Secure fax is easy with eFax Corporate (http://www.enterprise.efax.com). First, Just open a new email message.  In the To field, type your recipient's fax number @efaxsend.com. You can even send a secure international fax — just include t…
Polish reports in Access so they look terrific. Take yourself to another level. Equations, Back Color, Alternate Back Color. Write easy VBA Code. Tighten space to use less pages. Launch report from a menu, considering criteria only when it is filled…

762 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

18 Experts available now in Live!

Get 1:1 Help Now