Make error value a constant

memcall.go

@@ -1,7 +1,6 @@
 package memcall
 
 import (
-	"errors"
 	"runtime"
 )
 
@@ -33,7 +32,7 @@ func ReadWrite() MemoryProtectionFlag {
 }
 
 // ErrInvalidFlag indicates that a given memory protection flag is undefined.
-var ErrInvalidFlag = errors.New("<memguard::memcall> memory protection flag is undefined")
+const ErrInvalidFlag = "<memguard::memcall> memory protection flag is undefined"
 
 // Wipes a given byte slice.
 func wipe(buf []byte) {

memcall_freebsd.go

@@ -3,6 +3,7 @@
 package memcall
 
 import (
+	"errors"
 	"fmt"
 
 	"golang.org/x/sys/unix"
@@ -73,7 +74,7 @@ func Protect(b []byte, mpf MemoryProtectionFlag) error {
 	} else if mpf.flag == NoAccess().flag {
 		prot = unix.PROT_NONE
 	} else {
-		return ErrInvalidFlag
+		return errors.New(ErrInvalidFlag)
 	}
 
 	// Change the protection value of the byte slice.

memcall_openbsd.go

@@ -3,6 +3,7 @@
 package memcall
 
 import (
+	"errors"
 	"fmt"
 
 	"golang.org/x/sys/unix"
@@ -70,7 +71,7 @@ func Protect(b []byte, mpf MemoryProtectionFlag) error {
 	} else if mpf.flag == NoAccess().flag {
 		prot = unix.PROT_NONE
 	} else {
-		return ErrInvalidFlag
+		return errors.New(ErrInvalidFlag)
 	}
 
 	// Change the protection value of the byte slice.

memcall_osx.go

@@ -3,6 +3,7 @@
 package memcall
 
 import (
+	"errors"
 	"fmt"
 
 	"golang.org/x/sys/unix"
@@ -69,7 +70,7 @@ func Protect(b []byte, mpf MemoryProtectionFlag) error {
 	} else if mpf.flag == NoAccess().flag {
 		prot = unix.PROT_NONE
 	} else {
-		return ErrInvalidFlag
+		return errors.New(ErrInvalidFlag)
 	}
 
 	// Change the protection value of the byte slice.

memcall_test.go

@@ -42,7 +42,7 @@ func TestProtect(t *testing.T) {
 	if err := Protect(buffer, NoAccess()); err != nil {
 		t.Error(err)
 	}
-	if err := Protect(buffer, MemoryProtectionFlag{4}); err != ErrInvalidFlag {
+	if err := Protect(buffer, MemoryProtectionFlag{4}); err.Error() != ErrInvalidFlag {
 		t.Error("expected error")
 	}
 	Free(buffer)

memcall_unix.go

@@ -3,6 +3,7 @@
 package memcall
 
 import (
+	"errors"
 	"fmt"
 
 	"golang.org/x/sys/unix"
@@ -73,7 +74,7 @@ func Protect(b []byte, mpf MemoryProtectionFlag) error {
 	} else if mpf.flag == NoAccess().flag {
 		prot = unix.PROT_NONE
 	} else {
-		return ErrInvalidFlag
+		return errors.New(ErrInvalidFlag)
 	}
 
 	// Change the protection value of the byte slice.

memcall_windows.go

@@ -3,6 +3,7 @@
 package memcall
 
 import (
+	"errors"
 	"fmt"
 	"unsafe"
 
@@ -76,7 +77,7 @@ func Protect(b []byte, mpf MemoryProtectionFlag) error {
 	} else if mpf.flag == NoAccess().flag {
 		prot = 0x1 // PAGE_NOACCESS
 	} else {
-		return ErrInvalidFlag
+		return errors.New(ErrInvalidFlag)
 	}
 
 	var oldProtect uint32