[Webkit-unassigned] [Bug 50306] REGRESSION: WebChromeClient::shouldReplaceWithGeneratedFileForUpload() uses an initialized result value (breaks form submission with clang-built WebKit)

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=50306


Alexey Proskuryakov <ap at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|multipart/from-type forms   |REGRESSION:
                   |don't work in clang built   |WebChromeClient::shouldRepl
                   |WebKit                      |aceWithGeneratedFileForUplo
                   |                            |ad() uses an initialized
                   |                            |result value (breaks form
                   |                            |submission with clang-built
                   |                            |WebKit)
             Status|UNCONFIRMED                 |NEW
           Keywords|                            |NeedsRadar, Regression
           Priority|P2                          |P1
          Component|New Bugs                    |Forms
     Ever Confirmed|0                           |1




--- Comment #3 from Alexey Proskuryakov <ap at webkit.org>  2011-02-08 21:25:50 PST ---
I don't know how to build WebKit with clang, but I think that I know what's going on. We do call -[NSFileManager fileSystemRepresentationWithPath:] with a nil argument, it raises an exception, and we end up returning from -[_WebSafeForwarder forwardInvocation:] with an uninitialized result. 

This use of uninitialized result is a regression from shipping WebKit, I can observe it with a gcc build - even though the form doesn't fail to submit for me.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.