Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

gjson.New panic #3253

Closed
jdcloudconformance opened this issue Jan 9, 2024 · 1 comment · Fixed by #3255
Closed

gjson.New panic #3253

jdcloudconformance opened this issue Jan 9, 2024 · 1 comment · Fixed by #3255
Assignees
Labels
bug It is confirmed a bug, but don't worry, we'll handle it.

Comments

@jdcloudconformance
Copy link

What version of Go and system type/arch are you using?

go version go1.20.3 darwin/amd64

What version of GoFrame are you using?

github.com/gogf/gf/v2 v2.5.6

Can this bug be re-produced with the latest release?

What did you do?

image

What did you expect to see?

No error.

What did you see instead?

=== RUN TestXxx2
111--- FAIL: TestXxx2 (0.00s)
panic: reflect: call of reflect.Value.IsNil on string Value [recovered]
panic: reflect: call of reflect.Value.IsNil on string Value

@jdcloudconformance jdcloudconformance added the bug It is confirmed a bug, but don't worry, we'll handle it. label Jan 9, 2024
@jdcloudconformance
Copy link
Author

https://github.com/gogf/gf/blob/master/util/gconv/gconv_map.go

image

I tried the follow code:

image

The test code runs well.

oldme-git added a commit to oldme-git/gf that referenced this issue Jan 9, 2024
@oldme-git oldme-git mentioned this issue Jan 9, 2024
@oldme-git oldme-git self-assigned this Jan 10, 2024
gqcn pushed a commit that referenced this issue Jan 11, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug It is confirmed a bug, but don't worry, we'll handle it.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants