Return-Path: X-Original-To: apmail-incubator-callback-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-callback-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 59DF0DF67 for ; Tue, 9 Oct 2012 19:37:26 +0000 (UTC) Received: (qmail 27585 invoked by uid 500); 9 Oct 2012 19:37:26 -0000 Delivered-To: apmail-incubator-callback-dev-archive@incubator.apache.org Received: (qmail 27549 invoked by uid 500); 9 Oct 2012 19:37:26 -0000 Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: callback-dev@incubator.apache.org Delivered-To: mailing list callback-dev@incubator.apache.org Received: (qmail 27537 invoked by uid 99); 9 Oct 2012 19:37:26 -0000 Received: from tyr.zones.apache.org (HELO tyr.zones.apache.org) (140.211.11.114) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 09 Oct 2012 19:37:26 +0000 Received: by tyr.zones.apache.org (Postfix, from userid 65534) id A016F3CC49; Tue, 9 Oct 2012 19:37:25 +0000 (UTC) From: pmuellr To: callback-dev@incubator.apache.org Subject: incubator-cordova-ios pull request: CB-1600 - fix crash on iOS 4.3 in file ... Content-Type: text/plain Message-Id: <20121009193725.A016F3CC49@tyr.zones.apache.org> Date: Tue, 9 Oct 2012 19:37:25 +0000 (UTC) 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 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. ----