commands:sketch:sync Prevent panic if netrc is not configured

matteosuppo · matteosuppo · commit ca27f482fd44 · 2018-04-05T18:05:29.000+02:00
Signed-off-by: Matteo Suppo &lt;matteo.suppo@gmail.com&gt;
diff --git a/commands/sketch/sync.go b/commands/sketch/sync.go
@@ -448,7 +448,7 @@ func login() (string, string, error) {
 
 	logrus.Info("Searching for user credentials into the ~/.netrc file")
 	arduinoMachine := NetRC.FindMachine("arduino.cc")
-	if arduinoMachine.Name != "arduino.cc" {
+	if arduinoMachine == nil || arduinoMachine.Name != "arduino.cc" {
 		logrus.WithError(err).Error("Credentials not found")
 		return "", "", errors.New("Credentials not found, try typing `arduino login` to login")
 	}

Original file line number	Diff line number	Diff line change
`@@ -448,7 +448,7 @@ func login() (string, string, error) {`
`448`	`448`
`449`	`449`	`logrus.Info("Searching for user credentials into the ~/.netrc file")`
`450`	`450`	`arduinoMachine := NetRC.FindMachine("arduino.cc")`
`451`		`- if arduinoMachine.Name != "arduino.cc" {`
	`451`	`+ if arduinoMachine == nil \|\| arduinoMachine.Name != "arduino.cc" {`
`452`	`452`	`logrus.WithError(err).Error("Credentials not found")`
`453`	`453`	return "", "", errors.New("Credentials not found, try typing `arduino login` to login")
`454`	`454`	`}`