From: Russ Cox <rsc@golang.org>
Date: Sat, 5 Dec 2015 04:19:02 +0000 (-0500)
Subject: runtime: fix integer comparison in signal handling
X-Git-Tag: go1.6beta1~172
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=b9fde8605e1bc93d1a38d1165eedd297c910a3b2;p=gostls13.git

runtime: fix integer comparison in signal handling

(sig is unsigned, so sig-1 >= 0 is always true.)

Fixes #11281.

Change-Id: I4b9d784da6e3cc80816f2d2f7228d5d8a237e2d5
Reviewed-on: https://go-review.googlesource.com/17457
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
---

diff --git a/src/runtime/signal1_unix.go b/src/runtime/signal1_unix.go
index 3a8b5ba460..405713b371 100644
--- a/src/runtime/signal1_unix.go
+++ b/src/runtime/signal1_unix.go
@@ -228,11 +228,11 @@ func ensureSigM() {
 		for {
 			select {
 			case sig := <-enableSigChan:
-				if b := sig - 1; b >= 0 {
+				if b := sig - 1; sig > 0 {
 					sigBlocked[b/32] &^= (1 << (b & 31))
 				}
 			case sig := <-disableSigChan:
-				if b := sig - 1; b >= 0 {
+				if b := sig - 1; sig > 0 {
 					sigBlocked[b/32] |= (1 << (b & 31))
 				}
 			}