|
Home About Us A-Z Index Search * Contact Us Register Login Press ShopThe Open Brand -- Problem Reporting and Interpretations System |
Problem Report 2288 Details
Show help | Quick Search | Submit a Test Suite Support Request | Click here to view your privileges
This page provides all information on Problem Report 2288.
Report 2288 Actions
Problem Report Number 2288 Submitter's Classification Test Suite problem State Resolved Resolution Test Suite Deficiency (TSD) Problem Resolution ID TSD.X.1170 Raised 2003-10-30 18:02 Updated 2003-11-03 18:50 Published 2003-11-03 18:50 Product Standard X Window System Display Certification Program The Open Brand certification program Test Suite VSW version 5.1.3 Test Identification Xlib9/XCopyPlane 42 Problem Summary 32bit depth has been compiled in 64bit depth by mistake in cpypln.m Problem Text Turns out that the test program has a bug that only shows up with
thetarget framebuffer reports 32bit depth and the test is compiled in
64bitmode where a long is 64 bits and not 32 bits.
The developer of our team suggest a fix. I attached the fixed test
module and the code diffs here:
--- cpypln.m.orig Mon Oct 27 05:59:52 2003
+++ cpypln.m Tue Oct 28 15:29:55 2003
@@ -539,7 +539,7 @@
* If all planes are valid then a BadValue is not possible
* with this drawable
*/
- if (vp->depth >= CHAR_BIT*sizeof(long)) {
+ if (vp->depth >= CHAR_BIT*sizeof(CARD32)) {
trace("Visual with depth of %d, no planes invalid",
vp->depth);
unsupp++;
continue;Test Output *** Trace from Journal ****
400|1120 42 1 12:04:36|IC Start
200|1120 42 12:04:36|TP Start
520|1120 42 00010614 1 1|VSW5TESTSUITE PURPOSE 42
520|1120 42 00010614 1 2|Assertion XCopyPlane-42.(A)
520|1120 42 00010614 1 3|When the bit set in plane does not refer to a
valid pla
ne
520|1120 42 00010614 1 4|for the screen, then a BadValue error occurs.
520|1120 42 00010614 1 5|METH: For each drawable type
520|1120 42 00010614 1 6|METH: Ignore those that have no invalid planes.
520|1120 42 00010614 1 7|METH: Call XCopyPlane with plane set to first
invalid
number.
520|1120 42 00010614 1 8|METH: Verify that a BadValue error occurred.
520|1120 42 00010614 1 9|REPORT: Got Success, Expecting BadValue
220|1120 42 1 12:04:37|FAIL
410|1120 42 1 12:04:37|IC EndReview Information
Review Type TSMA Review Start Date 2003-10-30 18:02 Last Updated 2003-10-30 19:36 Completed 2003-10-30 19:36 Status Complete Review Recommendation Test Suite Deficiency (TSD) Review Response This is accepted as a fault in the test suite.
Note that the submitter's analysis of the problem, and suggested fix,
are incorrect. The problem is that the line:
plane = (1<<vp->depth);
can set plane to 1 (a valid value for plane) on LP64 architectures
when vp->depth is 32. The line should be:
plane = (1UL<<vp->depth);
Review Type SA Review Start Date 2003-10-30 19:36 Last Updated 2003-11-03 18:48 Completed 2003-11-03 18:48 Status Complete Review Resolution Test Suite Deficiency (TSD) Review Conclusion This is an agreed Test Suite Defiency.
Problem Reporting System Options:
- View Report 2288
- List All PRs
- Search Reports
- Email the System Administrator
- View the The Open Brand Interpretations Database User Manual
Contact the Certification Authority