Bug 15465 - iOS sample "Core Image" throws System.NullRefrenceException
Summary: iOS sample "Core Image" throws System.NullRefrenceException
Status: VERIFIED FIXED
Alias: None
Product: iOS
Classification: Xamarin
Component: Samples ()
Version: 7.0.3.x
Hardware: PC Mac OS
: High critical
Target Milestone: 7.0.3 (connection)
Assignee: Sebastien Pouliot
URL:
Depends on:
Blocks:
 
Reported: 2013-10-17 13:53 UTC by Mohit Kheterpal
Modified: 2013-10-18 11:55 UTC (History)
6 users (show)

Tags:
Is this bug a regression?: ---
Last known good build:

Notice (2018-05-24): bugzilla.xamarin.com is now in read-only mode.

Please join us on Visual Studio Developer Community and in the Xamarin and Mono organizations on GitHub to continue tracking issues. Bugzilla will remain available for reference in read-only mode. We will continue to work on open Bugzilla bugs, copy them to the new locations as needed for follow-up, and add the new items under Related Links.

Our sincere thanks to everyone who has contributed on this bug tracker over the years. Thanks also for your understanding as we make these adjustments and improvements for the future.


Please create a new report on Developer Community or GitHub with your current version information, steps to reproduce, and relevant error messages or log files if you are hitting an issue that looks similar to this resolved bug and you do not yet see a matching new report.

Related Links:
Status:
VERIFIED FIXED

Description Mohit Kheterpal 2013-10-17 13:53:36 UTC
Steps to reproduce:
1. Open CoreImage sample in XS.
2. Debug or run the application
3. Select Blur->GaussianBlur

Actual result:
Observe that application throws exception after selecting Any options in sample.

Supplement info: 
Exception detail: https://gist.github.com/AkhileshKumar01/9b2695602d3c57dfdfd2
iOS device log: https://gist.github.com/AkhileshKumar01/af26d29f51653dc5f52b

Expected result: Application should not crash

Environment details:
Mac Lion 10.7.4
MT 7.0.3.206
XS 4.0.13(build 39)
iPhone 6.1.3

App info: monotouch-samples-master 1bf501bb67021c9b3acdf043b19307e59e352a1f

Regression Status: REGRESSION, this issue working fine with stable MT 7.0.0.11
Comment 1 Sebastien Pouliot 2013-10-17 19:37:20 UTC
I get the NRE using iOS7 simulator. Will track the regression.
Comment 2 Sebastien Pouliot 2013-10-17 19:40:29 UTC
uho, that seems related to the addition of `IsDirectBinding = false` in
https://github.com/xamarin/maccore/commit/ff8f2ef7642d0be054abaf6880c3ebb18f003b25

c.c. Rolf

PJ, that pretty major and will need to be backported to 7.0.3 (.4)
Comment 3 Sebastien Pouliot 2013-10-17 19:56:13 UTC
Since we don't [Register] the CIFilter subclasses (they are not real types) then the `IsDirectBindings = false` on them is wrong.

That breaks the CIFilter defined selectors on them, including `OutputImage` property that results in the sample throwing NRE for every filter.
Comment 4 Sebastien Pouliot 2013-10-17 20:09:41 UTC
Fixed in master/b8f01028f0ebf70297a746c706571ec0976225b4

I'll ask for a 7.0.3-branch for maccore so we can backport this (without bringing additional commits).
Comment 5 Sebastien Pouliot 2013-10-18 08:44:35 UTC
7.0.3 fixed with 9121d5983cb70ae1b0f4ca2b2a470e58925c330d

7.0.4 fixed in edbf59df70e18736b4202dba83f0ece985e2810d
Comment 6 Sebastien Pouliot 2013-10-18 09:12:02 UTC
Unit test added in master 1c27cf6bd792111177c2365630806c30b4b8a262
Comment 7 narayanp 2013-10-18 10:49:59 UTC
Today I have checked this issue with following builds:

X.S 4.0.13(build 39)
X.iOS 7.0.3.208
iOS for VS 1.6.131

iOS Simulator 7.0

We are still getting exception when navigating sub-options under options. This is the exception details: https://gist.github.com/saurabh360/3b9dc7d32ce469371c53

Hence reopening this issue
Comment 8 Sebastien Pouliot 2013-10-18 11:07:14 UTC
Normal since 9121d5983cb70ae1b0f4ca2b2a470e58925c330d is 7.0.3.210. 
Any previous build will have the same issue.
Comment 9 PJ 2013-10-18 11:16:32 UTC
Thanks Lal. This bug should be fixed in the 7.0.3.210 build, it built an hour ago and is now in the dropbox. RESOLVING as FIXED
Comment 10 Atin 2013-10-18 11:55:41 UTC
We have checked this issue with following builds:

MT 7.0.3.210
XS 4.0.13(build 39)

Now its working fine Application is not crashing after selection option. Hence marking this as Verified Fixed