From: Rick Hudson <rlh@golang.org>
Date: Thu, 23 Apr 2015 20:28:15 +0000 (-0400)
Subject: runtime: Fix bug due to elided return.
X-Git-Tag: go1.5beta1~916
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=ada8cdb9f68f2e3441af1c514f8fbe28a2e07387;p=gostls13.git

runtime: Fix bug due to elided return.

A previous change to mbitmap.go dropped a return on a
path the seems not to be excersized. This was a mistake that
this CL fixes.

Change-Id: I715ee4ef08f5bf8d9f53cee84e8fb31a237e2d43
Reviewed-on: https://go-review.googlesource.com/9295
Reviewed-by: Austin Clements <austin@google.com>
---

diff --git a/src/runtime/mbitmap.go b/src/runtime/mbitmap.go
index f0704bdb5d..a1ebf03e69 100644
--- a/src/runtime/mbitmap.go
+++ b/src/runtime/mbitmap.go
@@ -187,6 +187,7 @@ func heapBitsForObject(p uintptr) (base uintptr, hbits heapBits, s *mspan) {
 			printunlock()
 			throw("objectstart: bad pointer in unexpected span")
 		}
+		return
 	}
 	// If this span holds object of a power of 2 size, just mask off the bits to
 	// the interior of the object. Otherwise use the size to get the base.