Ticket #999 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

r.mfilter.fp segfaults when divisor is 0

Reported by: juliocc Owned by: grass-dev@…
Priority: normal Milestone: 6.4.0
Component: Raster Version: unspecified
Keywords: mfilter segfault Cc:
Platform: Unspecified CPU: All

Description

r.mfilter.fp segfaults when the divisor is 0 because of an error in getfilt.c.

I'm attaching a patch that fixes this problem.

Attachments

getfilt.c.diff Download (347 bytes) - added by juliocc 2 years ago.
Fix

Change History

Changed 2 years ago by juliocc

Fix

in reply to: ↑ description   Changed 2 years ago by glynn

Replying to juliocc:

r.mfilter.fp segfaults when the divisor is 0 because of an error in getfilt.c.

Fixed in 7.0 with r41431.

  Changed 2 years ago by neteler

  • status changed from new to closed
  • resolution set to fixed

Fixed also in 6.4 and 6.5 (r41432 and r41433).

Thanks, juliocc.

Markus

  Changed 2 years ago by marisn

  • status changed from closed to reopened
  • resolution fixed deleted

This patch breaks 6.4 compilation. There's no "div" in 6.4 and 6.5. Markus - please revert it.

grass_64/raster/r.mfilter $ make
gcc -I/home/maris/soft/grass_64/dist.x86_64-unknown-linux-gnu/include  -g -O2       -DPACKAGE=\""grassmods"\"  -I/home/maris/soft/grass_64/dist.x86_64-unknown-linux-gnu/include -o OBJ.x86_64-unknown-linux-gnu/getfilt.o -c getfilt.c
getfilt.c: In function ‘get_filter’:
getfilt.c:46: warning: format not a string literal and no format arguments
getfilt.c:49: warning: format not a string literal and no format arguments
getfilt.c:71: warning: format not a string literal and no format arguments
getfilt.c:79: warning: format not a string literal and no format arguments
getfilt.c:82: warning: format not a string literal and no format arguments
getfilt.c:87: error: ‘div’ undeclared (first use in this function)
getfilt.c:87: error: (Each undeclared identifier is reported only once
getfilt.c:87: error: for each function it appears in.)
getfilt.c:99: warning: format not a string literal and no format arguments
getfilt.c:105: warning: format not a string literal and no format arguments
getfilt.c:108: warning: format not a string literal and no format arguments
getfilt.c:115: warning: format not a string literal and no format arguments
getfilt.c:122: warning: format not a string literal and no format arguments
getfilt.c:125: warning: format not a string literal and no format arguments
getfilt.c:155: warning: format not a string literal and no format arguments
make: *** [OBJ.x86_64-unknown-linux-gnu/getfilt.o] Error 1

  Changed 2 years ago by juliocc

The patch is against r.mfilter.fp. Revisions r41432, r41433 applied the patch to r.mfilter

  Changed 2 years ago by neteler

Sorry, done. I was tricked by the different module name.

Reverted previous and really fixed in 6.4 and 6.5 with r41438 and r41440.

  Changed 2 years ago by juliocc

  • status changed from reopened to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.