Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#6641 closed defect (fixed)

VLC hangs if "control=notify" is set in vlcrc

Reported by: Benjamin Drung Owned by: Rémi Denis-Courmont
Priority: normal Milestone: 2.0.4
Component: Interface: other plugins Version: 2.0.1
Severity: normal Keywords:
Cc: Difficulty: unknown
Platform(s): GNU/Linux Work status: Not started

Description

VLC freezes upon startup and the only recourse is to kill it, if following line is set in ~/.config/vlc/vlcrc:

control=notify:growl

It can be reproduces on Debian and Ubuntu and was reported on Debian: http://bugs.debian.org/662628

Change History (10)

comment:1 Changed 6 years ago by Edward Wang

Component: UnknownInterface: misc. plugins
Owner: changed from Jean-Baptiste Kempf to fkuehne, jb

What kind of freeze? Segfault freeze? Deadlock freeze? Anything interesting in the messages log?

comment:2 Changed 6 years ago by Benjamin Drung

Deadlock freeze. No interesting message on the terminal if vlc is launched with -vvv.

comment:3 Changed 6 years ago by Jean-Baptiste Kempf

Does it happen with growl alone?

comment:4 Changed 6 years ago by Jean-Baptiste Kempf

Milestone: 2.0.22.0.x maintenance bugs
Owner: changed from fkuehne, jb to Christophe Mutricy

comment:5 Changed 6 years ago by Benjamin Drung

No. It does happen with notify alone.

comment:6 Changed 6 years ago by Jean-Baptiste Kempf

Summary: VLC hangs if "control=notify:growl" is set in vlcrcVLC hangs if "control=notify" is set in vlcrc

comment:7 Changed 6 years ago by Divinainflationes

Happens to me too on Kubuntu 12.04 with updates with backports to date. control=telepathy:notify causes vlc to instafreeze without recovery and without further terminal output when I open any file, only sigkill gets rid of the process. When asked in IRC I got the advice to change the line to control=telepathy and everything is fine.

comment:8 Changed 5 years ago by Rémi Denis-Courmont

Milestone: 2.0.x maintenance bugs2.0.4
Owner: changed from Christophe Mutricy to Rémi Denis-Courmont
Status: newassigned

comment:9 Changed 5 years ago by Rémi Denis-Courmont

Platform(s): allGNU/Linux
Resolution: fixed
Status: assignedclosed

comment:10 Changed 5 years ago by remi@…

commit 1e5f4b465a82745e8e7e5a5de491deac39554ed3 Author: Rémi Denis-Courmont <remi@…> Date: Sun May 13 23:41:42 2012 +0300

Revert "Notify: add a callback to match telepathy, msn"

The "item-change" callback is called with the playlist lock held in at least one case: when the now playing meta is cleared while a new input is created by the playlist.

playlist_CurrentInput() acquires the playlist lock. So the "item-change" callback cannot call it. This bug is present in the Growl, MSN and Telepathy modules since version 1.0.0. It got copied into Notify in version 2.0.0.

This reverts commit fc56b92af1636c53a0545109a74476fe74c054a5. This should fix #6641.

Note: See TracTickets for help on using tickets.