Hello Matt:
Great!! Thanks again for the catch
Take care
Garrett
On Nov 18, 2005, at 11:49 AM, Matt Revelle wrote:
> Hi Garrett,
>
> Thanks, works good.
>
> Cheers,
> Matt
>
>
> On 11/18/05 11:25, "Garrett Potts" <potts@...> wrote:
>
>> Hello MAtt:
>>
>> I just checked a fix in. Could you test it for me please. We do
>> need the increment of 3 but it is in the wrong spot. It is suppose
>> to have been outside the block loop. The +3 increment is required
>> since we are going from a band that is indexed and creating 3
>> channels from it so it is a 1 to 3 band. Although I have not had any
>> it should support if multiple bands are indexed. So if the input are
>> 2 bands and each are indexed to 3 bands then you will have 6 band
>> out.
>>
>> Thank you for the catch!!!
>>
>>
>> Let me know if it's Ok or not
>>
>>
>> Take care
>>
>> Garrett
>> On Nov 18, 2005, at 11:05 AM, Matt Revelle wrote:
>>
>>> Hi all,
>>>
>>> It looks like there's a bug in ossimGdalTileSource.cpp (line 510)
>>> where the
>>> variable anOssimBandIndex is being incremented by 3. The line seems
>>> unnecessary since all 3 bands are being read at once, but it also
>>> introduces
>>> an error when multiple GDAL tiles need to be accessed to read the
>>> requested
>>> area.
>>>
>>> The first time loadIndexTo3BandTile is called, anOssimBandIndex has
>>> the
>>> correct value, 0, but the second time the value is 3 and will
>>> likely be an
>>> invalid index and result in a bad memory access.
>>>
>>> Commenting out the line appears to solve the problem.
>>>
>>> Cheers,
>>> Matt
>>>
>>>
>>>
>>>
>>> -------------------------------------------------------
>>> This SF.Net email is sponsored by the JBoss Inc. Get Certified
>>> Today
>>> Register for a JBoss Training Course. Free Certification Exam
>>> for All Training Attendees Through End of 2005. For more info visit:
>>> http://ads.osdn.com/?ad_id=7628&alloc_id=16845&op=click
>>> _______________________________________________
>>> http://www.ossim.org
>>> Ossim-developer mailing list
>>> Ossim-developer@...
>>> https://lists.sourceforge.net/lists/listinfo/ossim-developer
>>
>
>
>
>
> -------------------------------------------------------
> This SF.Net email is sponsored by the JBoss Inc. Get Certified Today
> Register for a JBoss Training Course. Free Certification Exam
> for All Training Attendees Through End of 2005. For more info visit:
> http://ads.osdn.com/?ad_id=7628&alloc_id=16845&op=click
> _______________________________________________
> http://www.ossim.org
> Ossim-developer mailing list
> Ossim-developer@...
> https://lists.sourceforge.net/lists/listinfo/ossim-developer
|