<pre style='margin:0'>
Joshua Root (jmroot) pushed a commit to branch master
in repository macports-base.
</pre>
<p><a href="https://github.com/macports/macports-base/commit/01a507a6811c21359271f5706332f13d4a30787d">https://github.com/macports/macports-base/commit/01a507a6811c21359271f5706332f13d4a30787d</a></p>
<pre style="white-space: pre; background: #F8F8F8"><span style='display:block; white-space:pre;color:#808000;'>commit 01a507a6811c21359271f5706332f13d4a30787d
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Sun Feb 5 17:46:19 2017 +1100
<span style='display:block; white-space:pre;color:#404040;'> Fix validity checks in ui_ask_singlechoice
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> Check that input is an integer before doing a comparison with it, and
</span><span style='display:block; white-space:pre;color:#404040;'> check that it is positive.
</span>---
src/port/port.tcl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/src/port/port.tcl b/src/port/port.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index f8aa9a0..8b30f71 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/src/port/port.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/src/port/port.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -5480,7 +5480,7 @@ namespace eval portclient::questions {
</span> throw
}
signal -restart error {TERM INT}
<span style='display:block; white-space:pre;background:#ffe0e0;'>- if {($input <= [llength $ports] && [string is wideinteger -strict $input])} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ if {[string is wideinteger -strict $input] && $input <= [llength $ports] && $input > 0} {
</span> return [expr {$input - 1}]
} else {
puts "Please enter an index from the above list."
</pre><pre style='margin:0'>
</pre>