From: <bug...@wi...> - 2003-01-30 17:23:18
|
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=104829 Changed by rb...@ro.... --- shadow/104829 Thu Jan 30 12:19:52 2003 +++ shadow/104829.tmp.28405 Thu Jan 30 12:19:53 2003 @@ -0,0 +1,29 @@ +Bug#: 104829 +Product: GStreamer +Version: 0.6.0 +OS: All +OS Details: RH8 +Status: NEW +Resolution: +Severity: normal +Priority: Normal +Component: gstreamer (core) +AssignedTo: gst...@bu... +ReportedBy: rb...@ro... +QAContact: gst...@bu... +TargetMilestone: 0.4.0 +URL: +Summary: crash when removing plugin without rerunning gst-register + +Any pipeline that uses a dynamic element (such as spider) which searches +the registry for elements to use will crash (segfault) if that element does +no longer exist (e.g. because it was removed). + +Example: +gst-register +rm $(libdir)/gstreamer-0.x/libgstwincodec.so +gst-launch filesrc location=bla ! spider ! xvideosink + +will segfault. Shouldn't this display a nice error message telling the user +he's a moron for removing elements and maybe even tell the user how he can +fix it (re-run gst-register)? |
From: <bug...@wi...> - 2003-05-13 04:37:39
|
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=104829 Changed by ds...@sc.... --- shadow/104829 Thu Jan 30 12:19:53 2003 +++ shadow/104829.tmp.6712 Tue May 13 00:37:35 2003 @@ -24,6 +24,9 @@ rm $(libdir)/gstreamer-0.x/libgstwincodec.so gst-launch filesrc location=bla ! spider ! xvideosink will segfault. Shouldn't this display a nice error message telling the user he's a moron for removing elements and maybe even tell the user how he can fix it (re-run gst-register)? + +------- Additional Comments From ds...@sc... 2003-05-13 00:37 ------- +Is this still a problem now that the registry is rebuilt? |
From: <bug...@wi...> - 2003-05-15 19:27:00
|
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=104829 Changed by rb...@ro.... --- shadow/104829 Tue May 13 00:37:35 2003 +++ shadow/104829.tmp.7406 Thu May 15 15:26:56 2003 @@ -27,6 +27,21 @@ will segfault. Shouldn't this display a nice error message telling the user he's a moron for removing elements and maybe even tell the user how he can fix it (re-run gst-register)? ------- Additional Comments From ds...@sc... 2003-05-13 00:37 ------- Is this still a problem now that the registry is rebuilt? + +------- Additional Comments From rb...@ro... 2003-05-15 15:26 ------- +Afaik, 0.6.1 had that patch included, and 0.6.1 still has the problem. + +[rbultje@shrek rbultje]$ gst-launch filesrc location=file.mp3 ! spider +! osssink +INFO ( 6726: 0) Initializing GStreamer Core Library version 0.6.1 +INFO ( 6726: 0) CPU features: (00000000) MMX SSE + +GStreamer-ERROR **: No default scheduler name - do you have a registry ? +aborting... +Aborted +[rbultje@shrek rbultje]$ + +I'll try 0.6.x CVS next. |
From: <bug...@wi...> - 2003-07-29 06:43:27
|
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=104829 Changed by ds...@sc.... --- shadow/104829 Thu May 15 15:26:56 2003 +++ shadow/104829.tmp.5940 Tue Jul 29 02:43:20 2003 @@ -42,6 +42,17 @@ GStreamer-ERROR **: No default scheduler name - do you have a registry ? aborting... Aborted [rbultje@shrek rbultje]$ I'll try 0.6.x CVS next. + +------- Additional Comments From ds...@sc... 2003-07-29 02:43 ------- + +This happens if + + - you have no user registry + - the system registry is non-writable (duh!) + - /usr/lib/gstreamer-0.6/ is modified + +basically, the code realizes it is supposed to update the registry, +but doesn't, since it can't write the system registry. |
From: <bug...@wi...> - 2003-07-29 06:58:57
|
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=104829 Changed by ds...@sc.... --- shadow/104829 Tue Jul 29 02:43:20 2003 +++ shadow/104829.tmp.11470 Tue Jul 29 02:58:49 2003 @@ -53,6 +53,9 @@ - you have no user registry - the system registry is non-writable (duh!) - /usr/lib/gstreamer-0.6/ is modified basically, the code realizes it is supposed to update the registry, but doesn't, since it can't write the system registry. + +------- Additional Comments From ds...@sc... 2003-07-29 02:58 ------- +The following patch may fix it (untested) |
From: <bug...@wi...> - 2003-07-29 06:59:28
|
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=104829 Changed by ds...@sc.... --- shadow/104829 Tue Jul 29 02:58:49 2003 +++ shadow/104829.tmp.11648 Tue Jul 29 02:59:20 2003 @@ -56,6 +56,11 @@ basically, the code realizes it is supposed to update the registry, but doesn't, since it can't write the system registry. ------- Additional Comments From ds...@sc... 2003-07-29 02:58 ------- The following patch may fix it (untested) + +------- Additional Comments From ds...@sc... 2003-07-29 02:59 ------- +Created an attachment (id=18709) +patch + |
From: <bug...@wi...> - 2003-07-29 09:01:52
|
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=104829 Changed by rb...@ro.... --- shadow/104829 Tue Jul 29 02:59:20 2003 +++ shadow/104829.tmp.8120 Tue Jul 29 05:01:45 2003 @@ -61,6 +61,10 @@ The following patch may fix it (untested) ------- Additional Comments From ds...@sc... 2003-07-29 02:59 ------- Created an attachment (id=18709) patch + +------- Additional Comments From rb...@ro... 2003-07-29 05:01 ------- +But why doesn't it create a user registry instead of crashing? Or is +that too obvious? |
From: <bug...@wi...> - 2003-07-29 18:48:01
|
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=104829 Changed by ds...@sc.... --- shadow/104829 Tue Jul 29 05:01:45 2003 +++ shadow/104829.tmp.26227 Tue Jul 29 14:26:45 2003 @@ -65,6 +65,13 @@ patch ------- Additional Comments From rb...@ro... 2003-07-29 05:01 ------- But why doesn't it create a user registry instead of crashing? Or is that too obvious? + +------- Additional Comments From ds...@sc... 2003-07-29 14:26 ------- +Because of the line removed in the patch, it never even gets to the +point where it attempts to rebuild the registry. If the file doesn't +exist, it is not readable, so it fails that test. It seems ok to +remove the line, since gst_registry_load() checks for readability (and +existence, etc.) as well. |
From: <bug...@wi...> - 2003-08-06 01:20:48
|
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=104829 Changed by ds...@sc.... --- shadow/104829 Tue Jul 29 14:26:45 2003 +++ shadow/104829.tmp.377 Tue Aug 5 21:20:41 2003 @@ -8,15 +8,15 @@ Severity: normal Priority: Normal Component: gstreamer (core) AssignedTo: gst...@bu... ReportedBy: rb...@ro... QAContact: gst...@bu... -TargetMilestone: 0.4.0 +TargetMilestone: 0.6.3 URL: -Summary: crash when removing plugin without rerunning gst-register +Summary: [0.6.3] crash when removing plugin without rerunning gst-register Any pipeline that uses a dynamic element (such as spider) which searches the registry for elements to use will crash (segfault) if that element does no longer exist (e.g. because it was removed). Example: @@ -72,6 +72,9 @@ ------- Additional Comments From ds...@sc... 2003-07-29 14:26 ------- Because of the line removed in the patch, it never even gets to the point where it attempts to rebuild the registry. If the file doesn't exist, it is not readable, so it fails that test. It seems ok to remove the line, since gst_registry_load() checks for readability (and existence, etc.) as well. + +------- Additional Comments From ds...@sc... 2003-08-05 21:20 ------- +I applied the patch to HEAD, and I think it works. |
From: <bug...@wi...> - 2003-08-26 10:43:02
|
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=104829 Changed by th...@ur.... --- shadow/104829 Tue Aug 5 21:20:41 2003 +++ shadow/104829.tmp.12343 Tue Aug 26 06:42:29 2003 @@ -8,13 +8,13 @@ Severity: normal Priority: Normal Component: gstreamer (core) AssignedTo: gst...@bu... ReportedBy: rb...@ro... QAContact: gst...@bu... -TargetMilestone: 0.6.3 +TargetMilestone: 0.6.4 URL: Summary: [0.6.3] crash when removing plugin without rerunning gst-register Any pipeline that uses a dynamic element (such as spider) which searches the registry for elements to use will crash (segfault) if that element does no longer exist (e.g. because it was removed). |
From: <bug...@wi...> - 2003-08-26 15:25:25
|
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=104829 Changed by th...@ur.... --- shadow/104829 Tue Aug 26 06:42:29 2003 +++ shadow/104829.tmp.7841 Tue Aug 26 11:24:54 2003 @@ -1,20 +1,20 @@ Bug#: 104829 Product: GStreamer Version: 0.6.0 OS: All OS Details: RH8 -Status: NEW -Resolution: +Status: RESOLVED +Resolution: FIXED Severity: normal Priority: Normal Component: gstreamer (core) AssignedTo: gst...@bu... ReportedBy: rb...@ro... QAContact: gst...@bu... -TargetMilestone: 0.6.4 +TargetMilestone: 0.6.3 URL: Summary: [0.6.3] crash when removing plugin without rerunning gst-register Any pipeline that uses a dynamic element (such as spider) which searches the registry for elements to use will crash (segfault) if that element does no longer exist (e.g. because it was removed). @@ -75,6 +75,9 @@ exist, it is not readable, so it fails that test. It seems ok to remove the line, since gst_registry_load() checks for readability (and existence, etc.) as well. ------- Additional Comments From ds...@sc... 2003-08-05 21:20 ------- I applied the patch to HEAD, and I think it works. + +------- Additional Comments From th...@ur... 2003-08-26 11:24 ------- +applied to 0.6.2.1, closed. |