Fix unhandled nil err check to prevent rpc causing a crash (#78)

sonic-net · Jul 28, 2021 · 085f29d · 085f29d
1 parent 6df988c
commit 085f29d
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/transl_utils/transl_utils.go b/transl_utils/transl_utils.go
@@ -23,8 +23,11 @@ func __log_audit_msg(ctx context.Context, reqType string, uriPath string, err er
     var err1 error
     username := "invalid"
     statusMsg := "failure"
+    errMsg := "None"
     if (err == nil) {
         statusMsg = "success"
+    } else {
+        errMsg = err.Error()
     }
 
     if Writer == nil {
@@ -38,7 +41,7 @@ func __log_audit_msg(ctx context.Context, reqType string, uriPath string, err er
     common_utils.GetUsername(ctx, &username)
 
     auditMsg := fmt.Sprintf("User \"%s\" request \"%s %s\" status - %s error - %s",
-                            username, reqType, uriPath, statusMsg, err.Error())
+                            username, reqType, uriPath, statusMsg, errMsg)
     Writer.Info(auditMsg)
 }