Thread Tools Display Modes
09-11-13, 05:02 PM   #1
lgwagner
A Defias Bandit
Join Date: Sep 2013
Posts: 2
Bug on changing talents and glyphs nUI v5.07.36(Release)

Wenever i try to change a talent or a glyph it shows a message and the glyphs and the talents dont change

Here is the message:

Date: 2013-09-11 19:51:35
ID: 1
Error occured in: AddOn: nUI
Count: 1
Message: Error: AddOn nUI attempted to call a forbidden function (RemoveTalent()) from a tainted execution path.
Debug:
[C]: RemoveTalent()
...rface\AddOns\Blizzard_TalentUI\Blizzard_TalentUI.lua:39: OnAccept()
..\FrameXML\StaticPopup.lua:3823: StaticPopup_OnClick()
[string "*:OnClick"]:1:
[string "*:OnClick"]:1
Locals:
None
AddOns:
Swatter, v4.4.0 (<%codename%>)
Babylonian, v5.1.DEV.332(/embedded)
Configator, v5.1.DEV.344(/embedded)
Gatherer, v4.4.0
GladiusEx, v
nUI, v5.07.36 (Lite)
SlideBar, v4.4.0 (<%codename%>)
BlizRuntimeLib_enUS v5.4.0.50400 <us>
(ck=f0)
 
09-11-13, 05:36 PM   #2
spiel2001
nUI's Author
 
spiel2001's Avatar
AddOn Author - Click to view addons
Join Date: Jun 2008
Posts: 7,724
Yeah... this is a known issue in nUI5. It's a taint error and related to how the Blizzard pulldown menus work. There's no easy fix for it. That said, it's been my experience that if you hold your nose just right, squint ever so slightly and only let your tongue stick out about a quarter inch from the corner of your mouth, it will work after a few tries. ~smile~ When all else fails, you might have to temporarily disable... which sucks, I know. I don't *think* this problem exists in nUI6... though, come to think of it, I'm not sure I've tested that fully.
__________________

What people don't get is that I am, ultimately, an artist at heart.
My brush has two colors, 1 and 0, and my canvas is made of silicon.



Official nUI Web Site: http://www.nUIaddon.com
Official nUI Support Forum: http://forums.nUIaddon.com
My day job: http://www.presidio.com/
 
09-11-13, 08:13 PM   #3
lgwagner
A Defias Bandit
Join Date: Sep 2013
Posts: 2
what is very strange is that on teh previous version for wow 5.3 it didint have this bug!
I really dont know what cause the addon to have this bug but what i know its that it didin't had it before!
 
09-11-13, 10:45 PM   #4
OromisGlaedr
A Defias Bandit
Join Date: Sep 2013
Posts: 3
I've been using nUI for some time now, and I haven't had this problem for this current patch, either. What could have happened?
 
09-11-13, 10:57 PM   #5
OromisGlaedr
A Defias Bandit
Join Date: Sep 2013
Posts: 3
Also, it does have that bug in nUI6 as well. Just tried.
 
09-12-13, 06:03 PM   #6
Belechannas
A Warpwood Thunder Caller
Join Date: May 2010
Posts: 86
This just started happening for me after 5.4 too.

I vaguely remember having this problem with another addon in the past. If memory serves, the only real fix is to remove all drop-down menus/items created by the addon - which may not be a showstopper for me (I don't *think* I regularly use any nUI-created dropdowns).
 
09-13-13, 04:25 PM   #7
Belechannas
A Warpwood Thunder Caller
Join Date: May 2010
Posts: 86
An update, but not a very happy one, I'm afraid.

I've spent hours trying to get to the bottom of this.

It does not appear to be related to pulldown menus. I've disabled them, and the variables that can be corrupted by pull down menus are not tainted.

I have gone through and removed dozens of instances in nUI where variables weren't declared local when they should have. No change.

Another issue which has caused mysterious taint errors in the past is corruption of the lua global variable "_" (underscore). Not tainted.

I have disabled nUI's management of tooltips, the minimap, and the battlefield minimap InfoPanel plugin (all of which were causing taint to non-nUI globals). Didn't fix the problem either.

The StaticPopup dialog boxes have caused a number of taint problems in the past. They too do not seem to be tainted.

After removing every logged source of taint I could find, my Taint.log file shows only two variables that are tainted that don't have "nUI" somewhere in the name. Those are

MOVING_CHAT_FRAME, and

BINDING_KEYS_CLICK

The first is used to move the chat windows and combat log around to where nUI wants them, and the second is used to handle key bindings. Disabling these would screw the addon up badly enough that it didn't really seem worth trying.

I could not find any changes in the Blizzard interface code that would account for the changed behavior, either. The error happens on a call to a C-routine.

Since there are cases where things get tainted without being logged, I also wrote a script to scan through the entire global namespace (_G) and report any non-tabular variables that were tainted and did not have "nui" (case-insensitive) in the name.

Apart from the two above, there were a few taints here:

Code:
9/13 15:10:12.609  Silmara says: EventTraceFrameEvent_DisplayTooltip
9/13 15:10:12.609  Silmara says: FrameStackTooltip_OnUpdate
9/13 15:10:12.609  Silmara says: EventTraceFrame_OnMouseWheel
9/13 15:10:12.609  Silmara says: ScriptErrorsFrame_DeleteError
9/13 15:10:12.609  Silmara says: FrameStackTooltip_Toggle
9/13 15:10:12.609  Silmara says: EventTraceFrame_RemoveEvent
9/13 15:10:12.609  Silmara says: DEBUGLOCALS_LEVEL
9/13 15:10:12.609  Silmara says: EventTraceFrame_OnShow
9/13 15:10:12.609  Silmara says: ScriptErrorsFrame_Update
9/13 15:10:12.609  Silmara says: EventTraceFrame_OnSizeChanged
9/13 15:10:12.609  Silmara says: EventTraceFrame_Update
9/13 15:10:12.625  Silmara says: EVENT_TRACE_EVENT_HEIGHT
9/13 15:10:12.625  Silmara says: EventTraceFrame_HandleSlashCmd
9/13 15:10:12.625  Silmara says: ScriptErrorsFrame_OnLoad
9/13 15:10:12.625  Silmara says: FACTION_UPDATE_END_STRING
9/13 15:10:12.625  Silmara says: DebugTooltip_OnLoad
9/13 15:10:12.625  Silmara says: ScriptErrorsFrameButton_OnClick
9/13 15:10:12.625  Silmara says: ScriptErrorsFrame_UpdateButtons
9/13 15:10:12.625  Silmara says: ScriptErrorsFrame_OnShow
9/13 15:10:12.625  Silmara says: EventTraceFrameEvent_OnLeave
9/13 15:10:12.625  Silmara says: EventTraceFrame_OnUpdate
9/13 15:10:12.625  Silmara says: EventTraceFrameEvent_OnEnter
9/13 15:10:12.625  Silmara says: EventTraceFrame_UpdateKeyboardStatus
9/13 15:10:12.625  Silmara says: EventTraceFrame_StopEventCapture
9/13 15:10:12.625  Silmara says: EventTraceFrame_OnEvent
9/13 15:10:12.625  Silmara says: EventTraceFrame_OnLoad
9/13 15:10:12.625  Silmara says: EVENT_TRACE_MAX_ENTRIES
9/13 15:10:12.625  Silmara says: FrameStackTooltip_OnEnter
9/13 15:10:12.625  Silmara says: ScriptErrorsFrame_OnError
9/13 15:10:12.625  Silmara says: EventTraceFrameEvent_OnClick
9/13 15:10:12.625  Silmara says: EventTraceFrame_StartEventCapture
9/13 15:10:12.625  Silmara says: FRAMESTACK_UPDATE_TIME
9/13 15:10:12.625  Silmara says: FrameStackTooltip_OnShow
9/13 15:10:12.625  Silmara says: _ERROR_COUNT
9/13 15:10:12.625  Silmara says: FACTION_UPDATE_INCREASE_STRING
9/13 15:10:12.625  Silmara says: FACTION_UPDATE_START_STRING
9/13 15:10:12.625  Silmara says: EventTraceFrame_OnKeyUp
9/13 15:10:12.641  Silmara says: EventTraceFrameEventHideButton_OnClick
Most of the tainted stuff seems to be related to diagnostic tools like eventtrace, framestack, etc. Nothing that would obviously interact with any part of the GlyphUI, certainly.

I didn't do a *recursive* scan through _G, which might turn up something. Or it might just crash the client if there are any circular references... Once I recover from my depression, I'll probably try.

I also attach my taint log, in case somebody smarter can find something I missed.
Attached Files
File Type: txt taint.log.txt (70.9 KB, 341 views)

Last edited by Belechannas : 09-13-13 at 04:27 PM.
 
 

WoWInterface » Featured Projects » nUI, MozzFullWorldMap and PartySpotter » Support » nUI: Bug Reports » Bug on changing talents and glyphs nUI v5.07.36(Release)

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off