Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 319582 - AIOOBE calling IAHyperlink methods
Summary: AIOOBE calling IAHyperlink methods
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.6.1   Edit
Assignee: Carolyn MacLeod CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-12 10:40 EDT by Carolyn MacLeod CLA
Modified: 2010-07-12 11:57 EDT (History)
1 user (show)

See Also:
Silenio_Quarti: review+

Attachments
test case that shows the failure (5.27 KB, text/plain)
2010-07-12 10:40 EDT, Carolyn MacLeod CLA 		no flags Details
patch to fix the problem (1.12 KB, patch)
2010-07-12 10:43 EDT, Carolyn MacLeod CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Carolyn MacLeod CLA 2010-07-12 10:40:37 EDT 
Created attachment 174033 [details]
test case that shows the failure

Run the attached snippet, which sets up a hyperlink listener with a name and description. Run AccProbe, and try to look at the IAHyperlink name and description. There is an array index out of bounds calling "method5" which is IAccessibleHyperlink::getDescription. The number of parameters is incorrect - it should be 2, but instead, there are 0.
Comment 1 Carolyn MacLeod CLA 2010-07-12 10:43:31 EDT 
Created attachment 174036 [details]
patch to fix the problem
Comment 2 Carolyn MacLeod CLA 2010-07-12 10:52:51 EDT 
The attached patch specifies the correct number of parameters for IAHyperlink methods. The problem was that when the IAAction method parameters were included in the IAHyperlink spec (because IAHyperlink derives from IAAction), QueryInterface, AddRef, and Release were included twice.

SSQ please review the patch for 3.6.1.
Comment 3 Carolyn MacLeod CLA 2010-07-12 11:57:47 EDT 
Fixed > 20100712 in 3.6.1 and in HEAD (3.7).