You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cordova.apache.org by pmuellr <gi...@git.apache.org> on 2012/10/09 21:37:25 UTC

incubator-cordova-ios pull request: CB-1600 - fix crash on iOS 4.3 in file ...

GitHub user pmuellr opened a pull request:

    https://github.com/apache/incubator-cordova-ios/pull/57

    CB-1600 - fix crash on iOS 4.3 in file transfer

    https://issues.apache.org/jira/browse/CB-1600
    
    Did two basic things here:
    
    1) add whitelist check before CDVFileTransfer#download/upload even start,
    to prevent the confusing behavior of whitelist checking in the middle of
    the download
    
    2) in the case where the whitelist passes, but gets one of the error
    conditions that we originally saw the crash in, add some extra protection
    to the cast in CDVFileTransfer#connection:didReceiveResponse:
    of NSURLResponse to NSHTTPURLResponse, since sometimes it isn't one.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/pmuellr/incubator-cordova-ios CB-1600

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-cordova-ios/pull/57.patch

----
commit 7589751f49ef22b4f6c973ab44f618967c5f034f
Author: Patrick Mueller <pm...@apache.org>
Date:   2012-10-09T12:23:52-07:00

    CB-1600 - fix crash on iOS 4.3 in file transfer
    
    https://issues.apache.org/jira/browse/CB-1600
    
    Did two basic things here:
    
    1) add whitelist check before CDVFileTransfer#download/upload even start,
    to prevent the confusing behavior of whitelist checking in the middle of
    the download
    
    2) in the case where the whitelist passes, but gets one of the error
    conditions that we originally saw the crash in, add some extra protection
    to the cast in CDVFileTransfer#connection:didReceiveResponse:
    of NSURLResponse to NSHTTPURLResponse, since sometimes it isn't one.

----