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.

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

digest: fix memory leak, fix not quoted 'opaque' #9199

Closed
wants to merge 1 commit into from
Closed

digest: fix memory leak, fix not quoted 'opaque' #9199

wants to merge 1 commit into from

Conversation

Karlson2k
Copy link
Contributor

Fix leak regression introduced by 3a6fe0c

emilengler
emilengler reviewed Jul 25, 2022
View reviewed changes
Copy link
Contributor

@emilengler emilengler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jay jay closed this in 6e241bb Jul 25, 2022
@jay
Copy link
Member

jay commented Jul 25, 2022

Thanks

@Karlson2k Karlson2k deleted the digest_auth_fix_04 branch July 25, 2022 19:54
sergio-nsk pushed a commit to snxd/curl that referenced this pull request Jul 27, 2022
@Karlson2k @sergio-nsk
digest: fix memory leak, fix not quoted 'opaque'
cf98280 
Fix leak regression introduced by 3a6fe0c.

Closes curl#9199
@sergio-nsk sergio-nsk mentioned this pull request Jul 27, 2022
Merged
sergio-nsk pushed a commit to snxd/curl that referenced this pull request Jul 27, 2022
@Karlson2k @sergio-nsk
digest: fix memory leak, fix not quoted 'opaque'
b5537cc 
Fix leak regression introduced by 3a6fe0c.

Closes curl#9199
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@emilengler emilengler emilengler left review comments

Assignees
No one assigned
Labels
authentication
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@Karlson2k @jay @emilengler @bagder