[GRASS-user] r.colors: stack smashing detected when large floating point numbers added to rules

GRASS GRASS user
1

Hi,

r.colors terminates if a large floating-point number is added to the rules list (specifically, 12 decimal places or more).

In spearfish:

$ r.colors map=elevation.10m color=rules
Enter rules, "end" when done, "help" if you need it.
fp: Data range is 1061.0640869999999722494976595 to 1846.74340800000004492176231

1061.064086999999 blue

*** stack smashing detected ***: r.colors terminated
Aborted (core dumped)

It's not crucial for me to be able to enter color rules this precisely, but r.colors should probably handle cases like these without crashing.

~ Eric.

2

Patton, Eric wrote:

r.colors terminates if a large floating-point number is added to the rules list (specifically, 12 decimal places or more).

In spearfish:

$ r.colors map=elevation.10m color=rules
Enter rules, "end" when done, "help" if you need it.
fp: Data range is 1061.0640869999999722494976595 to 1846.74340800000004492176231
>1061.064086999999 blue
*** stack smashing detected ***: r.colors terminated
Aborted (core dumped)

It's not crucial for me to be able to enter color rules this
precisely, but r.colors should probably handle cases like these
without crashing.

Fixed in CVS (buffers enlarged from 16 bytes to 80 bytes).

--
Glynn Clements <glynn@gclements.plus.com>

3

-----Original Message-----
From: Glynn Clements [mailto:glynn@gclements.plus.com]
Sent: Sat 6/2/2007 8:21 AM
To: Patton, Eric
Cc: grassuser@grass.itc.it
Subject: Re: [GRASS-user] r.colors: stack smashing detected when large floatingpoint numbers added to rules

Patton, Eric wrote:

r.colors terminates if a large floating-point number is added to the rules list (specifically, 12 decimal places or more).

In spearfish:

$ r.colors map=elevation.10m color=rules
Enter rules, "end" when done, "help" if you need it.
fp: Data range is 1061.0640869999999722494976595 to 1846.74340800000004492176231
>1061.064086999999 blue
*** stack smashing detected ***: r.colors terminated
Aborted (core dumped)

It's not crucial for me to be able to enter color rules this
precisely, but r.colors should probably handle cases like these
without crashing.

Fixed in CVS (buffers enlarged from 16 bytes to 80 bytes).

Thanks Glynn, very much appreciated.

~ Eric.