dwarf/line: remove foundFile "optimization" from LineToPC

There is no guarantee that files will end up stored contiguously in the
debug_line section which makes this optimization wrong in the general
case.
In particular with recent versions of go1.11 and a go.mod file present
the go compiler seems to sometimes produce executables that actually
violate this assumption.

pkg/dwarf/line/state_machine.go

@@ -264,10 +264,7 @@ func (lineInfo *DebugLineInfo) LineToPC(filename string, lineno int) uint64 {
 		return 0
 	}
 
-	var (
-		foundFile bool
-		sm        = newStateMachine(lineInfo, lineInfo.Instructions)
-	)
+	sm := newStateMachine(lineInfo, lineInfo.Instructions)
 
 	// if no instruction marked is_stmt is found fallback to the first
 	// instruction assigned to the filename:line.
@@ -280,12 +277,7 @@ func (lineInfo *DebugLineInfo) LineToPC(filename string, lineno int) uint64 {
 			}
 			break
 		}
-		if foundFile && sm.file != filename {
-			break
-		}
-		if sm.line == lineno && sm.file == filename {
-			foundFile = true
-			if sm.valid {
+		if sm.line == lineno && sm.file == filename && sm.valid {
 			if sm.isStmt {
 				return sm.address
 			} else if fallbackPC == 0 {
@@ -293,7 +285,6 @@ func (lineInfo *DebugLineInfo) LineToPC(filename string, lineno int) uint64 {
 			}
 		}
 	}
-	}
 	return fallbackPC
 }
 

pkg/proc/proc.go

@@ -640,7 +640,7 @@ func FirstPCAfterPrologue(p Process, fn *Function, sameline bool) (uint64, error
 		// breakpoint with file:line and with the function name always result on
 		// the same instruction being selected.
 		entryFile, entryLine := fn.cu.lineInfo.PCToLine(fn.Entry, fn.Entry)
-		if pc, _, err := p.BinInfo().LineToPC(entryFile, entryLine); err == nil {
+		if pc, _, err := p.BinInfo().LineToPC(entryFile, entryLine); err == nil && pc >= fn.Entry && pc < fn.End {
 			return pc, nil
 		}
 	}