bugGNU Astronomy Utilities - Bugs: bug #61007, Crash due to CFITSIO 4.0.0 version...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #61007: Crash due to CFITSIO 4.0.0 version format change

Submitter:  Mohammad Akhlaghi <makhlaghi>
Submitted:  Tue 03 Aug 2021 06:42:07 PM UTC
 
Category:  Installation Severity:  3 - Normal
Item Group:  * Crash Status:  Fixed
Privacy:  Public Assigned to:  makhlaghi
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       No canned response available

 

Tue 03 Aug 2021 07:26:00 PM UTC, comment #1: 

Fortunately the bug was easy to reproduce and it has been fixed in Commit 1db6add7.

Vincenzo and Zohreh, you can download the most recent tarball (that now contains this bug fix) and its corresponding book from the links below:

https://akhlaghi.org/gnuastro.pdf
https://akhlaghi.org/gnuastro-latest.tar.lz

I guess more people may confront this bug as the package managers upgrade their versions of CFITSIO, so hopefully I'll make a new release soon ;-).

But generally, many bugs have been found and fixed since version 0.15, you can see the full list of new features and bug fixes in the links above within Gnuastro's NEWS file.

Mohammad Akhlaghi <makhlaghi>
Project Administrator
Tue 03 Aug 2021 06:42:07 PM UTC, original submission:  

From version 4.0.0, CFITSIO has changed its version format. Before this, it was a 2 digit number (like 3.49 or 3.48 and etc), but from major version 4, it will have three digits! This change has been described in the What's new, as ("Changed version numbering to 3-field format").

Until now, Gnuastro was assuming the old format and thus printing the version as a floating point number. But this won't work for the new versioning format of CFITSIO (which is no longer a floating point number) and will cause a crash in Gnuastro:

fits.c: In function ‘gal_fits_key_write_version_in_ptr’:
fits.c:2106:36: error: too many decimal points in number
   sprintf(cfitsioversion, "%-.2f", CFITSIO_VERSION);
                                    ^
Makefile:2329: recipe for target 'fits.lo' failed

I am working on this and will post a comment when the fix is implemented.

This bug was reported to me today independently by Vincenzo Testa and Zohreh Ghaffari.

Mohammad Akhlaghi <makhlaghi>
Project Administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by makhlaghi (Submitted the item)
  • -email is unavailable- added by makhlaghi
  • -email is unavailable- added by makhlaghi
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

     

     

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2021-08-03 makhlaghi StatusIn Progress Fixed
        Open/ClosedOpen Closed
    2021-08-03 makhlaghi Carbon-Copy- Added -email is unavailable-
        Carbon-Copy- Added -email is unavailable-

    Back to the top


    Powered by Savane 3.9