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
Fix IPC::Connection::sendOutputMessage(IPC::UnixMessage&) Syscall param sendmsg(msg.msg_iov[2]) points to uninitialised byte(s) #2273
Fix IPC::Connection::sendOutputMessage(IPC::UnixMessage&) Syscall param sendmsg(msg.msg_iov[2]) points to uninitialised byte(s) #2273
Conversation
62c1e76
to
4f8abf1
Compare
@@ -52,7 +52,7 @@ class UpdateInfo { | |||
|
|||
// The size of the web view. | |||
WebCore::IntSize viewSize; | |||
float deviceScaleFactor; | |||
float deviceScaleFactor { 0 }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this not correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Normally we'd initialize a scale factor to 1, not 0.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should I change that in a new PR? I thought this was getting set elsewhere if used and just needed to be initialized to anything.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…am sendmsg(msg.msg_iov[2]) points to uninitialised byte(s) https://bugs.webkit.org/show_bug.cgi?id=242518 Reviewed by Fujii Hironori. Initialize deviceScaleFactor and updateScaleFactor so that we don't try to encode uninitialized memory. * Source/WebKit/Shared/UpdateInfo.h: Canonical link: https://commits.webkit.org/252330@main
4f8abf1
to
f4367f2
Compare
Committed 252330@main (f4367f2): https://commits.webkit.org/252330@main Reviewed commits have been landed. Closing PR #2273 and removing active labels. |
f4367f2