[Webkit-unassigned] [Bug 60875] [SOUP] Abnormal operation if server sends 5xx status code without HTTP body

[bugzilla-daemon at webkit.org]

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


Keunsoon Lee <keunsoon.lee at samsung.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #93736|0                           |1
        is obsolete|                            |
  Attachment #93736|commit-queue?               |
               Flag|                            |
  Attachment #94059|                            |review?, commit-queue?
               Flag|                            |




--- Comment #6 from Keunsoon Lee <keunsoon.lee at samsung.com>  2011-05-19 04:33:20 PST ---
Created an attachment (id=94059)
 --> (https://bugs.webkit.org/attachment.cgi?id=94059&action=review)
modify ChangeLog, add a new helper function, check handle->client() again

You can test the problematic situation on following test site.
    http://spe.mobilephone.net/wit/witguide.html
    WIT Root -> Common -> HTTP -> httperror.xhtml

If you are working with EFL port, you cannot find anything different between attached patch and original code.
Because EFL port does not support download flow and does not have its own error page.
But, you can see trying download on another port which is supporting download or showing own error page.

I also found 4xx status code cause same problem, so add it on a new patch.

And check handle->client() again after calling didReceiveResponse() on sendRequestCallback().
Because, WebCore makes handle->client() zero after didReceiveResponse() if something wrong on received response.
(I guess it means cancel.)
For example, received response's Content-Type is "foo/foo", which you can test on the test site.

Thank you.

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