This is a better fix (test added to avoid a crash if no camera/lens has
been selected):

diff --git a/ufraw_ufraw.c b/ufraw_ufraw.c
index 120e6a1..9eea022 100644
--- a/ufraw_ufraw.c
+++ b/ufraw_ufraw.c
@@ -1361,6 +1361,12 @@ int ufraw_flip_image(ufraw_data *uf, int flip)
     if ( uf->conf->threshold>0 )
        ufraw_flip_image_buffer(&uf->Images[ufraw_denoise_phase], flip);
     ufraw_flip_image_buffer(&uf->Images[ufraw_develop_phase], flip);
+#ifdef HAVE_LENSFUN
+    if (uf->modifier &&
+       (uf->postproc_ops & (LF_MODIFY_TCA | LF_MODIFY_DISTORTION |
+                             LF_MODIFY_GEOMETRY | LF_MODIFY_SCALE)))
+       ufraw_flip_image_buffer(&uf->Images[ufraw_lensfun_phase], flip);
+#endif
 
     return UFRAW_SUCCESS;
 }

It would be better though to factor the LF_* flags testing out: there are
too many tests for this already (with- or without LF_MODIFY_VIGNETTING).

Note that there still seems to be a minor bitrot issue in rotated +
flipped lensfun previews but no SIGSEGV anymore: I think it's a second bug,
also reproducable in stock 0.15

-- 
Frank

------------------------------------------------------------------------------
Enter the BlackBerry Developer Challenge  
This is your chance to win up to $100,000 in prizes! For a limited time, 
vendors submitting new applications to BlackBerry App World(TM) will have
the opportunity to enter the BlackBerry Developer Challenge. See full prize  
details at: http://p.sf.net/sfu/Challenge
_______________________________________________
ufraw-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ufraw-devel

Reply via email to