optimize

examples/patch_func.go

@@ -25,5 +25,5 @@ func main() {
 
 //go:noinline
 func heyHey() {
-	fmt.Println("fuck")
+	fmt.Println("fake")
 }

examples/patch_func_symbol.go

@@ -25,5 +25,5 @@ func main() {
 
 //go:noinline
 func heyHey() {
-	fmt.Println("fuck")
+	fmt.Println("fake")
 }

readme.md

@@ -1,12 +1,10 @@
 # SuperMonkey
 
-This lib is not production ready, please don't use
-
 ## Introduction
 
-Patch all functions including which are unexported
+Patch all functions without limits, including which are unexported
 
-**Warning** : please add -l to your gcflags, or it will panic.
+**Warning** : please add -l to your gcflags or add `//go:noinline` to func which you want to patch.
 
 ### patch private function
 
@@ -23,7 +21,7 @@ import (
 
 func main() {
 	fmt.Println("original function output:")
-	heyHey() // fuck
+	heyHey() // fake
 	fmt.Println()
 
 	sm.Patch("main", "", "heyHey", func() {
@@ -35,11 +33,11 @@ func main() {
 
 	sm.UnpatchAll()
 	fmt.Println("unpatch all, then output:")
-	heyHey() // fuck
+	heyHey() // fake
 }
 
 func heyHey() {
-	fmt.Println("fuck")
+	fmt.Println("fake")
 }
 ```
 
@@ -75,7 +73,7 @@ func main() {
 
 //go:noinline
 func heyHey() {
-	fmt.Println("fuck")
+	fmt.Println("fake")
 }
 
 ```

supermonkey.go

@@ -1,6 +1,7 @@
 package supermonkey
 
 import (
+	"fmt"
 	"os"
 	"reflect"
 	"strconv"
@@ -25,6 +26,12 @@ func Patch(pkgName, typeName, methodName string, patchFunc interface{}) {
 // PatchByFullSymbolName needs user to provide the full symbol path
 func PatchByFullSymbolName(symbolName string, patchFunc interface{}) {
 	addr := symbolTable[symbolName]
+	if addr == 0 {
+		fmt.Printf("The symbol is %v, and the patch target addr is 0, there may be 2 possible reasons\n", symbolName)
+		fmt.Println("	1. the function is inlined, please add //go:noinline to function comment or add -l to gcflags")
+		fmt.Println("	2. your input for symbolName or pkg/obj/method is wrong, check by using go tool nm {your_bin_file}")
+		panic("")
+	}
 	originalBytes := replaceFunction(addr, (uintptr)(getPtr(reflect.ValueOf(patchFunc))))
 	patchRecord[addr] = originalBytes
 }