Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
Date: Fri, 22 Aug 2014 12:36:11 +0000 (UTC)
From: "Gavin Shiels (JIRA)" <jira@apache.org>
To: issues@commons.apache.org
Message-ID: <JIRA.12736053.1408710932119.11807.1408710971985@arcas>
In-Reply-To: <JIRA.12736053.1408710932119@arcas>
References: <JIRA.12736053.1408710932119@arcas>
Subject: [jira] [Created] (IMAGING-140) ExifReWriter always writes EXIF
 segment before JFIF segment
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

Gavin Shiels created IMAGING-140:
------------------------------------

             Summary: ExifReWriter always writes EXIF segment before JFIF segment
                 Key: IMAGING-140
                 URL: https://issues.apache.org/jira/browse/IMAGING-140
             Project: Commons Imaging
          Issue Type: Bug
          Components: Format: JPEG
    Affects Versions: 1.x, 0.97, 0.94-incubator
         Environment: All
            Reporter: Gavin Shiels


ExifRewriter.writeSegmentsReplacingExif () always writes the EXIF segment as the first segment written to the output stream even when there is JFIF segment present.

The code checks for the existence of the JFIF segment and it looks like the intention was then to insert the EXIF segment at index 1 but it is instead always inserted at index 0, the variable 'index' is not used after being set to 1. 

'index' should be passed as the first parameter to the segments.add() call.

/* ----------------------------------------------------------------------- */
int index = 0;
JFIFPieceSegment firstSegment = (JFIFPieceSegment) segments.get(index);

if (firstSegment.marker == JFIFMarker)
    index = 1;
        
segments.add(0, new JFIFPieceSegmentExif(JPEG_APP1_Marker,
markerBytes, markerLengthBytes, newBytes));

/* ----------------------------------------------------------------------- */


--
This message was sent by Atlassian JIRA
(v6.2#6252)