bugGNU Astronomy Utilities - Bugs: bug #51476, Crashes on 32-bit and big-endian...

 
 

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

bug #51476: Crashes on 32-bit and big-endian systems

Submitter:  Mohammad Akhlaghi <makhlaghi>
Submitted:  Sat 15 Jul 2017 09:38:12 PM UTC
 
Category:  MakeProfiles Severity:  5 - Blocker
Item Group:  Crash Status:  Fixed
Privacy:  Public Assigned to:  makhlaghi
Open/Closed:  Closed

Sat 15 Jul 2017 10:11:51 PM UTC, comment #1: 

The problem was corrected and checked (on a 32-bit and 64-bit little-endian) system. It has been pushed to the main repo.

It will be submitted to Debian to see if it (hopefully) builds correctly on all architectures (except `sh4').

Mohammad Akhlaghi <makhlaghi>
Project Administrator
Sat 15 Jul 2017 09:38:12 PM UTC, original submission:  

Gnuastro 0.3.31 (fix to bug #51467) was approved and uploaded onto Debian a few hours ago and almost immediately several crashes were reported in the tests (they are still ongoing, but already 12 architectures have failed!):

https://buildd.debian.org/status/package.php?p=gnuastro

A summary of the results is:

armel:   radeccat.sh segfault
armhf:   radeccat.sh segfault
i386:    radeccat.sh segfault
mips:    all mkprof segfault
mipsel:  radeccat.sh segfault
s390x:   all mkprof no mode option
hppa:    all mkprof segfault
powerpc: all mkprof segfault
ppc64:   all mkprof no mode option
sh4:     memory problem in arithmetic library.
sparc64: all mkprof Bus error
x32:     radeccat.sh segfault

Except for the `sh4' architecture, the rest are all in MakeProfiles and appear related.

After trying the build on a 32-bit virtual machine, I was able to find the cause of the problem: the `mode' element of the `mkprofparams' (defined in MakeProfile's `main.h') was defined as uint8_t, but in the `ui_parse_coordinate_mode', it was being read as an `int'! Francly it is strange for me why this didn't cause a crash on my own system!?!

This maybe one of the problems with compilers becoming "smarter" (and hiding problems)!

Mohammad Akhlaghi <makhlaghi>
Project Administrator

 

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

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)
  •  

    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.

     

     

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-07-15 makhlaghi StatusIn Progress Fixed
        Open/ClosedOpen Closed

    Back to the top


    Powered by Savane 3.9