After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 762575 - avfvideosrc: Crashes with uncaught exception - wrong type for AVCaptureDevice.activeVideoMinFrameDuration
avfvideosrc: Crashes with uncaught exception - wrong type for AVCaptureDevice...
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Mac OS
: Normal blocker
: 1.7.90
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-02-24 02:51 UTC by Joe Gorse
Modified: 2016-02-25 09:38 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Change NSNumber/double to CMTime (935 bytes, patch)
2016-02-24 15:18 UTC, Joe Gorse
committed Details | Review

Description Joe Gorse 2016-02-24 02:51:50 UTC
From the GST_DEBUG log:
0:00:00.520884000 10995    0x10182fa30 WARN            avfvideosrc avfvideosrc.m:738:__30-[GstAVFVideoSrcImpl setCaps:]_block_invoke: An unexcepted error occured: The activeVideoMinFrameDuration passed is not supported by the device.  Use -activeFormat.videoSupportedFrameRateRanges to discover valid ranges.

The following AVCaptureDevice expects type NSValue:CMTime and is currently getting NSValue:double
  [device setValue:frame_duration_value forKey:@"activeVideoMinFrameDuration"];
Comment 2 Joe Gorse 2016-02-24 15:18:01 UTC
Created attachment 322249 [details] [review]
Change NSNumber/double to CMTime

This fixes the crash for me, though full functionality of video capture is not there yet.
Comment 3 Alessandro Decina 2016-02-25 03:32:07 UTC
The fix looks ok and works on iOS 9. Can you submit a git format-patch patch?

Also, what other issues do you have with capture?
Comment 4 Sebastian Dröge (slomo) 2016-02-25 06:58:42 UTC
Bug #762645 is probably the other issues
Comment 5 Sebastian Dröge (slomo) 2016-02-25 09:37:35 UTC
commit 2b63a88f26bdaa7bc9ade594f9f80e20f3b6afe2
Author: Joe Gorse <jhgorse@gmail.com>
Date:   Thu Feb 25 11:34:40 2016 +0200

    avfvideosrc: Frame durations as CTime to the API, not double
    
    Newer iOS seems to automatically convert, older iOS/OSX just crashes.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=762575