svf-tools 1.0.705 → 1.0.707

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "svf-tools",
-  "version": "1.0.705",
+  "version": "1.0.707",
   "description": "* <b>[TypeClone](https://github.com/SVF-tools/SVF/wiki/TypeClone) published in our [ECOOP paper](https://yuleisui.github.io/publications/ecoop20.pdf) is now available in SVF </b> * <b>SVF now uses a single script for its build. Just type [`source ./build.sh`](https://github.com/SVF-tools/SVF/blob/master/build.sh) in your terminal, that's it!</b> * <b>SVF now supports LLVM-10.0.0! </b> * <b>We thank [bsauce](https://github.com/bsauce) for writing a user manual of SVF ([link1](https://www.jianshu.com/p/068a08ec749c) and [link2](https://www.jianshu.com/p/777c30d4240e)) in Chinese </b> * <b>SVF now supports LLVM-9.0.0 (Thank [Byoungyoung Lee](https://github.com/SVF-tools/SVF/issues/142) for his help!). </b> * <b>SVF now supports a set of [field-sensitive pointer analyses](https://yuleisui.github.io/publications/sas2019a.pdf). </b> * <b>[Use SVF as an external lib](https://github.com/SVF-tools/SVF/wiki/Using-SVF-as-a-lib-in-your-own-tool) for your own project (Contributed by [Hongxu Chen](https://github.com/HongxuChen)). </b> * <b>SVF now supports LLVM-7.0.0. </b> * <b>SVF now supports Docker. [Try SVF in Docker](https://github.com/SVF-tools/SVF/wiki/Try-SVF-in-Docker)! </b> * <b>SVF now supports [LLVM-6.0.0](https://github.com/svf-tools/SVF/pull/38) (Contributed by [Jack Anthony](https://github.com/jackanth)). </b> * <b>SVF now supports [LLVM-4.0.0](https://github.com/svf-tools/SVF/pull/23) (Contributed by Jared Carlson. Thank [Jared](https://github.com/jcarlson23) and [Will](https://github.com/dtzWill) for their in-depth [discussions](https://github.com/svf-tools/SVF/pull/18) about updating SVF!) </b> * <b>SVF now supports analysis for C++ programs.</b> <br />",
   "main": "index.js",
   "scripts": {

package/svf/include/Util/ExtAPI.json

@@ -518,8 +518,16 @@
     "asprintf": {
         "return":  "int",
         "arguments":  "(char **restrict, const char *restrict, ...)",
-        "type": "EFT_NOOP",
-        "overwrite_app_function": 1
+        "type": "EFT_A0R_NEW",
+        "overwrite_app_function": 1,
+        "AddrStmt": {
+            "src": "Obj",
+            "dst": "Dummy"
+        },
+        "StoreStmt": {
+            "src": "Dummy",
+            "dst": "Arg0"
+        }
     },
     "atexit":   {
         "return":  "int",
@@ -2192,8 +2200,16 @@
     "vasprintf":    {
         "return":  "int",
         "arguments":  "(char **, const char *, va_list)",
-        "type": "EFT_NOOP",
-        "overwrite_app_function": 1
+        "type": "EFT_A0R_NEW",
+        "overwrite_app_function": 1,
+        "AddrStmt": {
+            "src": "Obj",
+            "dst": "Dummy"
+        },
+        "StoreStmt": {
+            "src": "Dummy",
+            "dst": "Arg0"
+        }
     },
     "vfprintf": {
         "return":  "int",

package/svf-llvm/lib/SVFIRBuilder.cpp

@@ -1224,6 +1224,7 @@ void SVFIRBuilder::parseOperations(std::vector<ExtAPI::Operation>  &operations,
             else
             {
                 s32_t nodeIDType = ExtAPI::getExtAPI()->getNodeIDType(s);
+                // return value >= 0 is an argument node
                 if (nodeIDType >= 0)
                 {
                     if( svfcall->arg_size() <= (u32_t) nodeIDType)
@@ -1239,29 +1240,41 @@ void SVFIRBuilder::parseOperations(std::vector<ExtAPI::Operation>  &operations,
                         nodeIDMap[s] = pag->getValueNode(svfcall->getArgOperand(nodeIDType));
                     }
                 }
+                // return value = -1 is an inst node
                 else if (nodeIDType == -1)
                 {
                     operands.push_back(pag->getValueNode(svfcall));
                     nodeIDMap[s] = pag->getValueNode(svfcall);
                 }
+                // return value = -2 is a Dummy node
                 else if (nodeIDType == -2)
                 {
                     operands.push_back(pag->addDummyValNode());
                     nodeIDMap[s] = operands[operands.size() - 1];
                 }
+                // return value = -3 is an object node
                 else if (nodeIDType == -3)
                 {
                     if (svfcall->getType()->isPointerTy())
                     {
-                        operands.push_back(pag->getObjectNode(svfcall));
-                        nodeIDMap[s] = pag->getObjectNode(svfcall);
+                        NodeID objId;
+                        // Indirect call
+                        if (getCallee(svfcall) == nullptr)
+                            objId = pag->addDummyObjNode(svfcall->getType());
+                        else // Direct call
+                            objId = pag->getObjectNode(svfcall);
+
+                        operands.push_back(objId);
+                        nodeIDMap[s] = objId;
                     }
                 }
+                // return value = -4 is a nullptr node
                 else if (nodeIDType == -4)
                 {
                     operands.push_back(pag->getNullPtr());
                     nodeIDMap[s] = operands[operands.size() - 1];
                 }
+                // return value = -5 is an offset
                 else if (nodeIDType == -5)
                 {
                     for (char const &c : s)
@@ -1272,6 +1285,7 @@ void SVFIRBuilder::parseOperations(std::vector<ExtAPI::Operation>  &operations,
                     operands.push_back(atoi(s.c_str()));
                     nodeIDMap[s] = atoi(s.c_str());
                 }
+                // return value = -6 is an illegal operand format
                 else
                     assert(false && "The operand format of function operation is illegal!");
             }