Add name of function to message.

lcartey · lcartey · commit 7e5ef5f8306f · 2024-01-30T09:55:42.000Z
diff --git a/cpp/common/src/codingstandards/cpp/rules/functionerroneousreturnvaluenottested/FunctionErroneousReturnValueNotTested.qll b/cpp/common/src/codingstandards/cpp/rules/functionerroneousreturnvaluenottested/FunctionErroneousReturnValueNotTested.qll
@@ -58,5 +58,5 @@ query predicate problems(FunctionCall fc, string message) {
   not exists(GuardCondition gc |
     DataFlow::localFlow(DataFlow::exprNode(fc), DataFlow::exprNode(gc.getAChild*()))
   ) and
-  message = "Return value is not tested for errors."
+  message = "Return value from " + fc.getTarget().getName() + " is not tested for errors."
 }

Original file line number	Diff line number	Diff line change
`@@ -58,5 +58,5 @@ query predicate problems(FunctionCall fc, string message) {`
`58`	`58`	`not exists(GuardCondition gc \|`
`59`	`59`	`DataFlow::localFlow(DataFlow::exprNode(fc), DataFlow::exprNode(gc.getAChild*()))`
`60`	`60`	`) and`
`61`		`- message = "Return value is not tested for errors."`
	`61`	`+ message = "Return value from " + fc.getTarget().getName() + " is not tested for errors."`
`62`	`62`	`}`