Skip to content

Commit

Permalink
fix pointer conflict cause go panic, php-fpm will crash.
Browse files Browse the repository at this point in the history
	nginx-error.log will show: [error] 12199#0: *471 recv() failed (104: Connection reset by peer) while reading response header from upstream,
	kitech#8

Signed-off-by: monkey <nangong92t@gmail.com>
  • Loading branch information
nangong92t committed Aug 12, 2016
1 parent 9a7f3f4 commit c9bc5af
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions phpgo/extension.go
Original file line number Diff line number Diff line change
Expand Up @@ -426,6 +426,7 @@ func on_phpgo_function_callback_p(cbid int, phpthis unsafe.Pointer,
panic("wtf")
}
gothis := gext.objs_p[phpthis]
delete(gext.objs_p, phpthis)
// argv = append([]reflect.Value{reflect.ValueOf(gothis)}, argv...)
argv[0] = reflect.ValueOf(gothis)
}
Expand Down

0 comments on commit c9bc5af

Please sign in to comment.