[Gstreamer-bugs] [Bug 101678] Changed - "gnome-sound-recorder -r" segfaults without a filename.

bugzilla-daemon at widget.gnome.org bugzilla-daemon at widget.gnome.org
Fri Apr 4 13:33:02 PST 2003


Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

http://bugzilla.gnome.org/show_bug.cgi?id=101678

Changed by in7y118 at public.uni-hamburg.de.

--- shadow/101678	Fri Apr  4 13:03:40 2003
+++ shadow/101678.tmp.11663	Fri Apr  4 16:33:02 2003
@@ -156,6 +156,21 @@
 more important fix - invalid use of g_error in gst_init
 
 
 ------- Additional Comments From rbultje at ronald.bitfreak.net  2003-04-04 13:03 -------
 We should never use exit() inside a library. Just return or use
 g_error() or so. Imo, using g_error here is fine.
+
+------- Additional Comments From in7y118 at public.uni-hamburg.de  2003-04-04 16:33 -------
+1) gst_init parses command line parameters.
+
+2) gst_init only returns if it could initialize safely (currently that
+includes parsing all command line parameters). That's according to the
+docs.
+
+3) g_error is "A convenience function/macro to log an error message.
+Error messages are always fatal, resulting in a call to abort() to
+terminate the application. This function will result in a core dump;
+don't use it for errors you expect. Using this function indicates a
+bug in your program, i.e. an assertion failure."
+And no, I don't want a SIGSEGV when somebody passed a wrong argument
+to a gst program.





More information about the Gstreamer-bugs mailing list