[dev_finder] return error correctly when mdns start fails
The current error handling doesn't work because sending to `errChan`
blocks forever.
Bug: DX-931
Change-Id: I93be4735515446fda256d6c0e635e695590135e3
diff --git a/cmd/dev_finder/common.go b/cmd/dev_finder/common.go
index 4ba3f3d..0a6f5f1 100644
--- a/cmd/dev_finder/common.go
+++ b/cmd/dev_finder/common.go
@@ -114,7 +114,7 @@
ctx, cancel := context.WithTimeout(ctx, time.Duration(cmd.timeout)*time.Millisecond)
defer cancel()
if err := m.Start(ctx, cmd.mdnsPort); err != nil {
- errChan <- fmt.Errorf("starting mdns: %v", err)
+ return nil, fmt.Errorf("starting mdns: %v", err)
}
m.Send(packet)
devices := make([]*fuchsiaDevice, 0)
