]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit
Subject: Fix range validation of integer commands with "unlimited".
authorPedro Alves <palves@redhat.com>
Fri, 22 Mar 2013 20:25:40 +0000 (20:25 +0000)
committerPedro Alves <palves@redhat.com>
Fri, 22 Mar 2013 20:25:40 +0000 (20:25 +0000)
commit82b821e99da6e5360fcb86adf8edf73fd463240c
treee51fd6ab37ad73d2530d2a270bcba66c8ce86fdb
parentab19f3254b64b17b7e5fed11cc89b65e8330146e
Subject: Fix range validation of integer commands with "unlimited".

The range validation added by

  http://sourceware.org/ml/gdb-patches/2013-03/msg00767.html

Changes things to allow setting the command to INT_MAX or UINT_MAX
directly, with signed and unsigned commands respectively.  However,
that went a little bit too far, as in the cases of var_integer and
var_uinteger, those values are actually implementation detail.  It's
better to not expose them in the interface, and have users assume
those values mean "unlimited" too, so to be safer to expand the range
of the commands in the future if we want to.  Yes, it's pedantic, and
it's not likely users actually will do this, but MI frontends and
Python scripts might.

gdb/
2013-03-22  Pedro Alves  <palves@redhat.com>
    Yao Qi  <yao@codesourcery.com>
    Mark Kettenis  <kettenis@gnu.org>

* cli/cli-setshow.c (do_set_command) <var_uinteger>:
Don't let the user set the value to UINT_MAX directly.
<var_integer>: Don't let the user set the value to INT_MAX
directly.
gdb/ChangeLog
gdb/cli/cli-setshow.c