Bug 167701
Summary: | Fix for XmList out of bound accesses | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Red Hat Enterprise Linux 4 | Reporter: | Bastien Nocera <bnocera> | ||||||||||||
Component: | openmotif | Assignee: | Thomas Woerner <twoerner> | ||||||||||||
Status: | CLOSED ERRATA | QA Contact: | |||||||||||||
Severity: | medium | Docs Contact: | |||||||||||||
Priority: | medium | ||||||||||||||
Version: | 4.0 | CC: | tao | ||||||||||||
Target Milestone: | --- | ||||||||||||||
Target Release: | --- | ||||||||||||||
Hardware: | All | ||||||||||||||
OS: | Linux | ||||||||||||||
Whiteboard: | |||||||||||||||
Fixed In Version: | RHBA-2006-0292 | Doc Type: | Bug Fix | ||||||||||||
Doc Text: | Story Points: | --- | |||||||||||||
Clone Of: | Environment: | ||||||||||||||
Last Closed: | 2006-08-10 21:16:46 UTC | Type: | --- | ||||||||||||
Regression: | --- | Mount Type: | --- | ||||||||||||
Documentation: | --- | CRM: | |||||||||||||
Verified Versions: | Category: | --- | |||||||||||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||||||||
Cloudforms Team: | --- | Target Upstream Version: | |||||||||||||
Embargoed: | |||||||||||||||
Bug Depends On: | |||||||||||||||
Bug Blocks: | 180356, 181409 | ||||||||||||||
Attachments: |
|
Description
Bastien Nocera
2005-09-07 12:14:50 UTC
Created attachment 118552 [details]
openmotif-list-overrun2.patch
Created attachment 118553 [details]
xmbug.c
Click on one element in the list (without releasing the mouse button), then
release the button when the item has been removed from the list.
This usually causes a segfault.
Created attachment 118554 [details]
xmrotate.c
Select multiple items and extend the selection from back to top and vice-versa,
a segfault should arise soon after multiple extensions or selections.
Note that this change is on purpose. start is decreased by one when called as an argument of SelectRange. start == 0 would cause the index in SelectRange to be -1... @@ -4683,7 +4693,7 @@ SelectElement(Widget wid, SelectRange(lw, item, end + 1, sel); else if ((i > end) && (i <= start)) RestoreRange(lw, end, i - 1, FALSE); - else if (i > start) + else if (i > start && start > 0) { if (sel) SelectRange(lw, end, start - 1, FALSE); Created attachment 123873 [details]
New overrun patch.
Analyzing the patch and the code, I have fixed the code in a different way.
Please find attached the new patch.
Created attachment 123976 [details]
openMotif-2.2.3-overrun.patch
Revised version from ICS (upstream) with an additional fix.
An advisory has been issued which should help the problem described in this bug report. This report is therefore being closed with a resolution of ERRATA. For more information on the solution and/or where to find the updated files, please follow the link below. You may reopen this bug report if the solution does not work for you. http://rhn.redhat.com/errata/RHBA-2006-0292.html |