WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED WORKSFORME
252169
_web_capitalizeRFC822HeaderFieldName leaks memory
https://bugs.webkit.org/show_bug.cgi?id=252169
Summary
_web_capitalizeRFC822HeaderFieldName leaks memory
Seija K.
Reported
2023-02-13 07:45:29 PST
We do not free the backing buffer because we pass nullptr as the last argument to the noCopy functions and never explicitly deallocate.
Attachments
Add attachment
proposed patch, testcase, etc.
Seija K.
Comment 1
2023-02-13 07:48:14 PST
Pull request:
https://github.com/WebKit/WebKit/pull/10039
Darin Adler
Comment 2
2023-02-13 08:37:33 PST
How did you notice this bug? Code inspection or some type of testing. Could you supply more context to your work? The WebKit project requires regression tests when we fix bugs. Maybe we can forgo that in this case but I would like to know more about how it was discovered to make that decision.
Darin Adler
Comment 3
2023-02-13 09:41:07 PST
If we made a test, we’d see there’s no bug here. We do free the backing buffer. Passing nullptr as the last argument tells it to use the default allocator to deallocate.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug