Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mumble crashing when loading non-included DLLs #1004

Closed
mumble-voip opened this issue Jun 26, 2013 · 7 comments
Closed

Mumble crashing when loading non-included DLLs #1004

mumble-voip opened this issue Jun 26, 2013 · 7 comments

Comments

@mumble-voip
Copy link
Collaborator

Hello, I am working on a plugin, and something strange seems to be happening. It seems that mumble is crashing when it does not recognize the DLL name (ie. if I call my plugin cs.dll, mumble won't crash, but calling it unknown.dll causes mumble to crash). I'm using Windows 8

Steps to reproduce

  1. Compile positional audio plugin (can post my source code if necessary)
  2. Place in C:\Program Files (x86)\Mumble\plugins named unknown.dll
  3. Open Mumble
  4. Open Settings, instant crash
  5. Go back into plugins folder
  6. Rename unknown.dll to cs.dll (A default plugin, which I have moved out of the plugins folder)
  7. Open Mumble
  8. Open Settings, no crash

Seems weird to me. Using MumblePAHelper shows that my plugin is reporting the correct values, correct description, etc. even when not using an included plugin name, leading me to believe that this is something wrong with mumble itself. Thanks in advanced for your consideration!

This ticket has been migrated from sourceforge. It is thus missing some details like original creator etc.
The original is at https://sourceforge.net/p/mumble/bugs/1004/ .

The following attachments were added on the original item:

@mumble-voip
Copy link
Collaborator Author

Also, it may be worth it to note that the name change/removal of cs.dll doesn't actually seem to remove it, as it is still listed under the plugin list. Nor does changing my plugin name to cs.dll cause it to show up under the plugin list. Mumble also seems to function fine aside from opening the settings.

EDIT: The plugin also functions, it's only the settings screen that causes any crashing.

@mumble-voip
Copy link
Collaborator Author

A friend and I are experiencing the same problem. I believe that the plug-in is not loading in the reporter's description but rather mumble is re-downloading the original plug-in and overwriting the user's plugin (when the name is set to one of the stock plugin names).

I am also on Windows 8 running mumble 1.2.4. A friend experiences the same crash, I am uncertain what his mumble version / OS version are.

@mkrautz
Copy link
Contributor

mkrautz commented Jul 1, 2013

Mumble's Windows builds are built with MSVC10 (Visual Studio 2010), with the Windows 7 SDK. To ensure 100% C++ ABI compatibility, you should use the same compiler - otherwise you will most likely get crashes like is described here. :-)

Can you confirm that using MSVC10 fixes the issue for you? Also, as Micah points out, if you call your plugin 'cs.dll', Mumble is probably using its own 'cs.dll' (either by re-downloading, or by using the one distributed with hte installer) rather than yours.

@mumble-voip
Copy link
Collaborator Author

I'm using MSVC10 as my IDE and using the most recent Windows SDK (http://msdn.microsoft.com/en-us/library/windows/desktop/hh852363.aspx). I'll try to compile it with the Windows 7 SDK when I get home to check. Thanks for replying!

@mumble-voip
Copy link
Collaborator Author

Also, yes I've come to realize that my plugin is being replaced by Mumble's own when using a stock DLL name

@mumble-voip
Copy link
Collaborator Author

Alright, sorry for the triple post, but I just compiled it against the Windows 7 SDK and still I am getting crashes.

@Kissaki
Copy link
Member

Kissaki commented Nov 5, 2013

As you indicated you removed cd.dll but the plugin was still listed I guess the issue is this:

On installation Mumble puts the installers plugin DLLs into the installation folders plugins sub directory.
However, on plugin updates the new DLLs will be put into the user writable %APPDATA%\Mumble\Plugins folder.
So I guess you removed and replaced the cs.dll in your installation directory, but actually saw no crashes because the cs.dll that was used was still the original in the appdata folder.
Is this conclusion correct?

@Kissaki Kissaki closed this as completed Dec 16, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants