diff options
| author | Max Kellermann <max@musicpd.org> | 2020-10-28 17:19:08 +0100 |
|---|---|---|
| committer | Max Kellermann <max@musicpd.org> | 2020-10-28 17:21:27 +0100 |
| commit | 37710195ca581f3362e5db4ce3d5e4bdd2b1897e (patch) | |
| tree | f31b7d6feb3f96b51eab9ba993c6f60a53993c18 | |
| parent | 7b9295ff99f366fec9ba2ea0a12ed17b05b0af7c (diff) | |
meson_options.txt: disable the "smbclient" plugin by default
The bug https://bugzilla.samba.org/show_bug.cgi?id=11413 makes MPD
crash after at most a minute of using the plugin. Since this bug is
five years old already and it doesn't look like it will ever be fixed,
all libsmbclient code in MPD is scheduled for removal. For now, the
plugin is disabled by default so people are less likely to hit the
crash bug.
Closes https://github.com/MusicPlayerDaemon/MPD/issues/991
| -rw-r--r-- | NEWS | 1 | ||||
| -rw-r--r-- | doc/plugins.rst | 5 | ||||
| -rw-r--r-- | doc/user.rst | 2 | ||||
| -rw-r--r-- | meson_options.txt | 6 |
4 files changed, 12 insertions, 2 deletions
@@ -4,6 +4,7 @@ ver 0.22.2 (not yet released) on update * input - qobuz/tidal: fix protocol errors due to newlines in error messages + - smbclient: disable by default due to libsmbclient crash bug * playlist - soundcloud: fix protocol errors due to newlines in error messages * state_file: save on shutdown diff --git a/doc/plugins.rst b/doc/plugins.rst index e5056a7ab..107140942 100644 --- a/doc/plugins.rst +++ b/doc/plugins.rst @@ -71,6 +71,11 @@ Load music files from a SMB/CIFS server. It is used when :code:`music_directory` contains a ``smb://`` URI, for example :samp:`smb://myfileserver/Music`. +Note that :file:`libsmbclient` has a serious bug which causes MPD to +crash, and therefore this plugin is disabled by default and should not +be used until the bug is fixed: +https://bugzilla.samba.org/show_bug.cgi?id=11413 + nfs --- diff --git a/doc/user.rst b/doc/user.rst index 00ef9bdeb..db87ee7d0 100644 --- a/doc/user.rst +++ b/doc/user.rst @@ -86,7 +86,7 @@ For example, the following installs a fairly complete list of build dependencies libpulse-dev libshout3-dev \ libsndio-dev \ libmpdclient-dev \ - libnfs-dev libsmbclient-dev \ + libnfs-dev \ libupnp-dev \ libavahi-client-dev \ libsqlite3-dev \ diff --git a/meson_options.txt b/meson_options.txt index a1c5ec1ed..36214692d 100644 --- a/meson_options.txt +++ b/meson_options.txt @@ -92,7 +92,11 @@ option('cdio_paranoia', type: 'feature', description: 'libcdio_paranoia input pl option('curl', type: 'feature', description: 'HTTP client using CURL') option('mms', type: 'feature', description: 'MMS protocol support using libmms') option('nfs', type: 'feature', description: 'NFS protocol support using libnfs') -option('smbclient', type: 'feature', description: 'SMB support using libsmbclient') + +# The "smbclient" plugin is disabled by default because libsmbclient +# has a serious bug which crashes MPD very quickly: +# https://bugzilla.samba.org/show_bug.cgi?id=11413 +option('smbclient', type: 'feature', value: 'disabled', description: 'SMB support using libsmbclient') # # Commercial services |