Changeset 944

Timestamp:

Feb 14, 2007 1:45:08 AM (17 years ago)

Author:

bennylp

Message:

Fixed bug in Python module's normalize_stun_config(), and added Makefile for building Python module

Location:

pjproject/trunk/pjsip-apps/src/py_pjsua

Files:

• Makefile (added)
• py_pjsua.c (modified) (2 diffs)

pjproject/trunk/pjsip-apps/src/py_pjsua/py_pjsua.c

@@ -3131 +3131 @@
     PyObject * tmpObj;
     stun_config_Object *obj;
-    pjsua_stun_config *cfg;
+    pjsua_stun_config cfg;
 
     if (!PyArg_ParseTuple(pArgs, "O", &tmpObj))
@@ -3137 +3137 @@
         return NULL;
     }
-    if (tmpObj != Py_None)
-    {
-        obj = (stun_config_Object *) tmpObj;
-        cfg = (pjsua_stun_config *)malloc(sizeof(pjsua_stun_config));
-        cfg->stun_port1 = obj->stun_port1;
-        cfg->stun_port2 = obj->stun_port2;
-        cfg->stun_srv1.ptr = PyString_AsString(obj->stun_srv1);
-        cfg->stun_srv1.slen = strlen(PyString_AsString(obj->stun_srv1));
-        cfg->stun_srv2.ptr = PyString_AsString(obj->stun_srv2);
-        cfg->stun_srv2.slen = strlen(PyString_AsString(obj->stun_srv2));
-    } else {
-        cfg = NULL;
-    }
-    pjsua_normalize_stun_config(cfg);
-    obj->stun_port1 = cfg->stun_port1;
-    obj->stun_port2 = cfg->stun_port2;
+
+    if (tmpObj == Py_None) {
+        Py_INCREF(Py_None);
+        return Py_None;
+    }
+
+    obj = (stun_config_Object *) tmpObj;
+    cfg.stun_port1 = obj->stun_port1;
+    cfg.stun_port2 = obj->stun_port2;
+    cfg.stun_srv1.ptr = PyString_AsString(obj->stun_srv1);
+    cfg.stun_srv1.slen = strlen(PyString_AsString(obj->stun_srv1));
+    cfg.stun_srv2.ptr = PyString_AsString(obj->stun_srv2);
+    cfg.stun_srv2.slen = strlen(PyString_AsString(obj->stun_srv2));
+
+    pjsua_normalize_stun_config(&cfg);
+    obj->stun_port1 = cfg.stun_port1;
+    obj->stun_port2 = cfg.stun_port2;
     Py_XDECREF(obj->stun_srv1);
     obj->stun_srv1 = 
-        PyString_FromStringAndSize(cfg->stun_srv1.ptr, cfg->stun_srv1.slen);
+        PyString_FromStringAndSize(cfg.stun_srv1.ptr, cfg.stun_srv1.slen);
     Py_XDECREF(obj->stun_srv2);
     obj->stun_srv2 = 
-        PyString_FromStringAndSize(cfg->stun_srv2.ptr, cfg->stun_srv2.slen);
-    if (cfg != NULL)
-    {
-        free(cfg);
-    }
+        PyString_FromStringAndSize(cfg.stun_srv2.ptr, cfg.stun_srv2.slen);
+
     Py_INCREF(Py_None);
     return Py_None;