Merge pull request #2 from ItaloCobains/1-fix-code-scanning-alert-ret…

…urning-stack-allocated-memory Fix code scanning alert - Returning stack-allocated memory bug
ItaloCobains · Nov 14, 2023 · b127be1 · b127be1
2 parents 0dfffd8 + 1f40367
commit b127be1
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@italocobains/graph",
-  "version": "1.1.8",
+  "version": "1.1.9",
   "description": "This is a lib write by c++ and nodejs addon api for graph algorithm.",
   "main": "index.js",
   "scripts": {

diff --git a/src/graph.cc b/src/graph.cc
@@ -139,7 +139,8 @@ Edge *Graph::getEdge(int sourceId, int targetId) const
     {
         if (edge.sourceId == sourceId && edge.targetId == targetId)
         {
-            return &edge;
+            Edge *edgeCopy = new Edge(edge);
+            return edgeCopy;
         }
     }
     return nullptr;