Bug 179549
Summary: | Pasting into XmTextField is unreliable | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Red Hat Enterprise Linux 4 | Reporter: | Ron Yorston <rmy> | ||||||||||
Component: | openmotif | Assignee: | Thomas Woerner <twoerner> | ||||||||||
Status: | CLOSED ERRATA | QA Contact: | |||||||||||
Severity: | medium | Docs Contact: | |||||||||||
Priority: | medium | ||||||||||||
Version: | 4.0 | CC: | pgraner, poelstra | ||||||||||
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:17:04 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: | 180362, 181409, 405941 | ||||||||||||
Attachments: |
|
Description
Ron Yorston
2006-02-01 11:23:38 UTC
Created attachment 123954 [details]
Proposed patch
Created attachment 123975 [details]
Revised patch after code review.
This patch honors the text.max_char_size for generation of insert_orig in the
same way as ModifyVerify does.
Thanks for the revised patch, Thomas. I've tried it with my test program and it seems to work nicely. Valgrind is now happy in both UTF-8 and C locales. On further investigation I find that previous patches don't work if the clipboard selection is used rather than the primary selection (for example, because XmTextFieldPaste has been called). In this case ds->value (TextFSel.c line 1102) isn't null terminated. I've made a new patch to cover this case. Created attachment 124259 [details]
Revised patch to handle clipboard
Please attach your test case. Created attachment 124266 [details]
testcase
Here's a hacked version of one of the examples from the O'Reilly Motif book
that should illustrate the problem. I've changed the XmText widget in the
original to XmTextField.
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 |