@mattn, what version of Windows are you using?

Windows10 64bit

On 12/6/15, Russ Cox notifications@github.com wrote:

@mattn, what version of Windows are you using?

---

Reply to this email directly or view it on GitHub:
#13500 (comment)

• Yasuhiro Matsumoto

I think this works as expected. Your C program's output is the same as Go - it returns ERROR_FILE_NOT_FOUND.

I don't know why the RegOpenKeyEx fails on your computer. But I have seen all sorts of errors when reading all registry keys of various computers. People run all sorts of programs they download from Internet - buggy and shifty. These programs are free to do whatever they like with your registry - they have all your rights (more often then not administrator rights).

So I do not consider failing TestWalkFullRegistry to be a problem. I have added the test while I was developing package functionality, and left it in because it is helpful sometimes to debug things. But it is only running during "go test -short", so we ignore any registry inconsistencies.

Please, reopen if you disagree.

Alex

I do not have a strong opinion. It's a mental issue that all of the test is not passed. Also I am guessing that it is enough that you don't need to enumerate all of the registry.

For the memo or knowledge. There are another problem to run this test case on non-clean environment. This is not objection to current tests.

1. CLASSES_ROOT\VirtualStore\ require UAC to read

So should be ignored.

diff --git a/src/internal/syscall/windows/registry/registry_test.go b/src/internal/syscall/windows/registry/registry_test.go
index f63248c..564a126 100644
--- a/src/internal/syscall/windows/registry/registry_test.go
+++ b/src/internal/syscall/windows/registry/registry_test.go
@@ -10,6 +10,7 @@ import (
    "bytes"
    "crypto/rand"
    "os"
+   "strings"
    "syscall"
    "testing"
    "unsafe"
@@ -578,7 +579,7 @@ func walkKey(t *testing.T, k registry.Key, kname string) {
        func() {
            subk, err := registry.OpenKey(k, name, registry.ENUMERATE_SUB_KEYS|registry.QUERY_VALUE)
            if err != nil {
-               if err == syscall.ERROR_ACCESS_DENIED {
+               if err == syscall.ERROR_ACCESS_DENIED || strings.HasPrefix(kname, `CLASSES_ROOT\VirtualStore\`) {
                    // ignore error, if we are not allowed to access this key
                    return
                }

2. There are some broken registry values.

One of example, HKEY_CURRENT_USER\Software\PostgreSQL\StackBuilder

Those values seems to be stored as wide characters.