[Bug 734623] facedetect: fix NULL error when disable eyes/nose/mouth

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Mon Aug 11 12:10:41 PDT 2014


https://bugzilla.gnome.org/show_bug.cgi?id=734623
  GStreamer | gst-plugins-bad | git

Thiago Sousa Santos <thiagossantos> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #283128|none                        |needs-work
             status|                            |

--- Comment #3 from Thiago Sousa Santos <thiagossantos at gmail.com> 2014-08-11 19:10:36 UTC ---
Review of attachment 283128:
 --> (https://bugzilla.gnome.org/review?bug=734623&attachment=283128)

I would prefer this to be handled directly in the _load_profile function. Just
check for NULL there and return NULL in that case.

Additionally your patch seems to be agains gstfacedetect_orig.c, not against
gstfacedetect.c

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list