Skip to content
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

CDVURLSchemeHandler.m startURLSchemeTask data can be nil #903

Closed
3 tasks done
msmtamburro opened this issue Jun 17, 2020 · 1 comment · Fixed by #1035
Closed
3 tasks done

CDVURLSchemeHandler.m startURLSchemeTask data can be nil #903

msmtamburro opened this issue Jun 17, 2020 · 1 comment · Fixed by #1035
Milestone

Comments

@msmtamburro
Copy link
Contributor

Bug Report

Problem

A potentially nil value is passed to a method with a non-null specifier.

What is expected to happen?

didReceiveData should not be called with a nil value

What does actually happen?

nil can be passed

Information

This is a quick fix: just wrap line 78 with an if statement.

Command or Code

Environment, Platform, Device

Version information

Checklist

  • I searched for existing GitHub issues
  • I updated all Cordova tooling to most recent version
  • I included all the necessary information above
@dpogue dpogue added this to the 6.1.1 milestone Jun 22, 2020
NiklasMerz added a commit to GEDYSIntraWare/cordova-ios that referenced this issue Dec 2, 2020
@NiklasMerz
Copy link
Member

Thank you for this report. This is indeed a problem I just submitted a pull request to fix this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants