Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

vet warning: possible misuse of unsafe.Pointer #43

Open
nathany opened this issue Mar 8, 2019 · 3 comments
Open

vet warning: possible misuse of unsafe.Pointer #43

nathany opened this issue Mar 8, 2019 · 3 comments
Labels
bug help wanted

Comments

@nathany
Copy link
Contributor

nathany commented Mar 8, 2019

./wrap.go:165:53: possible misuse of unsafe.Pointer
@nathany
Copy link
Contributor Author

nathany commented Jan 19, 2022 •
edited

wrap.go#L132 possible misuse of reflect.StringHeader

wrap.go#L166 possible misuse of unsafe.Pointer

@nathany nathany pinned this issue Jan 19, 2022
@nathany nathany added the bug label Jan 19, 2022
@pbnjay
Copy link
Contributor

pbnjay commented Jan 19, 2022

Any insight on why these are there to begin with? The StringHeader replacement seems straightforward, but is there a leak it's otherwise accounting for?

@nathany
Copy link
Contributor Author

nathany commented Jan 19, 2022

No idea 🤷🏼

@arp242 arp242 unpinned this issue Aug 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nathany @pbnjay