style: fix naked return

free5gc · Nov 24, 2023 · 0cdbfa6 · 0cdbfa6
1 parent 89bfa61
commit 0cdbfa6
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/internal/context/user_plane_information.go b/internal/context/user_plane_information.go
@@ -748,7 +748,7 @@ func getPathBetween(cur *UPNode, dest *UPNode, visited map[*UPNode]bool,
 		path = make([]*UPNode, 0)
 		path = append(path, cur)
 		pathExist = true
-		return
+		return path, pathExist
 	}
 
 	selectedSNssai := selection.SNssai
@@ -760,16 +760,14 @@ func getPathBetween(cur *UPNode, dest *UPNode, visited map[*UPNode]bool,
 				continue
 			}
 
-			path_tail, path_exist := getPathBetween(node, dest, visited, selection)
+			path_tail, pathExist := getPathBetween(node, dest, visited, selection)
 
-			if path_exist {
+			if pathExist {
 				path = make([]*UPNode, 0)
 				path = append(path, cur)
-
 				path = append(path, path_tail...)
-				pathExist = true
 
-				return
+				return path, pathExist
 			}
 		}
 	}

diff --git a/internal/sbi/consumer/nnrf.go b/internal/sbi/consumer/nnrf.go
@@ -184,7 +184,7 @@ func SendNFDiscoveryPCF() (problemDetails *models.ProblemDetails, err error) {
 		logger.ConsumerLog.Warnln("handler returned wrong status code ", httpResp.Status)
 		if httpResp.Status != localErr.Error() {
 			err = localErr
-			return
+			return problemDetails, err
 		}
 		problem := localErr.(openapi.GenericOpenAPIError).Model().(models.ProblemDetails)
 		problemDetails = &problem