View Issue Details

IDProjectCategoryView StatusLast Update
0005381MMW v4Reports/Scriptspublic2009-05-09 12:02
Reporterrusty Assigned To 
PriorityimmediateSeverityminorReproducibilitysometimes
Status closedResolutionfixed 
Product Version3.1 
Target Version3.1Fixed in Version3.1 
Summary0005381: Uninstalling a script that doesn't exist leads to AV
DescriptionIf you rename Magic Nodes vbs script to xxx.vbs.off and then run MediaMonkey and try to uninstall the script (Tools > Extensions > Unistall), it triggers an AV.

Debug log sent automatically moments ago.
TagsNo tags attached.
Attached Files
ScreenShot.JPG (71,195 bytes)   
ScreenShot.JPG (71,195 bytes)   
Fixed in build1228

Activities

Ludek

2009-03-14 12:07

developer   ~0017104

Last edited: 2009-03-14 12:09

I tried this, but no AV occured to me, only the error dialog that the file '...Scripts/Auto/MagicNodes.vbs' cannot be opened (see the attached screenshot). If I clicked [Ignore] then all seemed to be uninstalled. This error dialog is all rigth, because the file really doesn't exist and is required in the'.../All Users/App Data/MediaMonkey/Extensions/MagicNodes/Uninstall.vbs' script.
So I don't know what could I fix in this area.

rusty

2009-03-17 13:19

administrator   ~0017128

I just tested this out again on a different machine and the error occurs exactly the same as previously.

Here are the repro steps:
1) install magic nodes
2) restart mm
3) rename magic nodes to magicnodes.vbs.off
4) from extensions menu try uninstalling magic nodes
-->AV

ELog was sent automatically (labelled with bug 5381)

Ludek

2009-03-17 19:34

developer   ~0017133

Assigned to Petr in order to find the Eureka log since I cannot reproduce this.

Ludek

2009-03-17 21:23

developer   ~0017134

Last edited: 2009-03-17 21:24

Rusty,
I see, you clicked [Cancel] instead of the [Ignore] on the "file cannot be opened" dialog and therefore there was raised the 'Action cancelled by user' exception.

Ok, I fixed this so that open file exceptions are not raised in case of scripts, but hopefully this will not cause a trobles in other cases.
i.e. now, only the dialog appears, but the cancel exception is not raised in case of clicking [Cancel] button.

Fixed in build 1228.

peke

2009-05-09 12:02

developer   ~0017771

Verified 1244