Pano13: struct Image initialization error

Bug #678763 reported by tksharpless
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Panotools
Fix Released
Medium
Unassigned

Bug Description

The Pano13 source in the hugin SDK has a typo in line 849 of filter.c, in function SetImageDefaults(). It reads
  bzero(im->formatParam, sizeof(im->precomputedValue));
but should be
  bzero(im->precomputedValue, sizeof(im->precomputedValue));

This leaves 4 of the precomputed values uninitialized since formatParam starts 4 words before precomputedValue and overwrites precomputedCount with (another) zero.

Unlikely to cause malfunction.

Revision history for this message
rew (r-e-wolff) wrote :

Just checked the source, and except for a few lines movement the bug is still there.

tags: added: libpano patch source
Changed in hugin:
importance: Low → Medium
status: New → Confirmed
Revision history for this message
rew (r-e-wolff) wrote :

yuv, You comitted this to the source repository (with someone elses name as the patch provider). Or did that someone else push my patch to the repository resulting in your attribution?

Revision history for this message
rew (r-e-wolff) wrote :

If "this is the one" can you move this to "fix comitted"?

Revision history for this message
Yuv (yuv) wrote :

Hi Rogier,
I don't know what you mean above. I just checked out the libpano source from SVN, noticed that the error was still there, applied your patch and committed. This bug report will have to be moved to Panotools once that bug tracker is online on Launchpad

Changed in hugin:
status: Confirmed → Fix Committed
Revision history for this message
Yuv (yuv) wrote :

P.S.: SVN1305

Yuv (yuv)
affects: hugin → panotools
tmodes (tmodes)
Changed in panotools:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.