Bug 220417

Summary: Loading of SGI images is broken due to bad endian swap code
Product: [Fedora] Fedora Reporter: Scott A. Friedman <friedman>
Component: DevILAssignee: Ian Chapman <packages>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: 6   
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-01-02 21:46:07 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:
Attachments:
Description Flags
patch to fix borked endian byteswapping none

Description Scott A. Friedman 2006-12-21 03:51:48 UTC
Description of problem:
This used to work just fine - but someone broke it along the way by trying to be
clever. Anyway, attached patch uses internal linux byteswap functionality when
requested in place of broken 'asm' 

Version-Release number of selected component (if applicable):
1.6.8-0.10.rc2

How reproducible:
Try loading an SGI image file - you will get an INVALID HEADER error because the
header values are not converted from big endian to little endian (when required)

Steps to Reproduce:
1.
2.
3.
  
Actual results:
Image file doesn't load

Expected results:
Image file should load!

Additional info:
I can attach an sgi rgb file for testing - but the patch I included works as
long as it is added to the spec file.

Comment 1 Scott A. Friedman 2006-12-21 03:51:48 UTC
Created attachment 144159 [details]
patch to fix borked endian byteswapping

Comment 2 Ian Chapman 2007-01-02 21:46:07 UTC
Thanks for patch. It's been added to the next version which should appear in the
repo shortly.