Fixed NULL-pointer bug when unloading of GSM interfaces
[lcr.git] / interface.c
index a4bac2d..ee753a9 100644 (file)
@@ -318,23 +318,28 @@ static int inter_portname(struct interface *interface, char *filename, int line,
        struct interface_port *ifport, **ifportp;
        struct interface *searchif;
 
-       /* check for port already assigned */
+       /* goto end of chain */
+       ifport = interface->ifport;
+       if (ifport) {
+               while(ifport->next)
+                       ifport = ifport->next;
+       }
+
+       /* check for port already assigned, but not for shared loop interface */
        searchif = interface_newlist;
-       while(searchif) {
-               ifport = searchif->ifport;
-               while(ifport) {
-                       if (!strcasecmp(ifport->portname, value)) {
-                               SPRINT(interface_error, "Error in %s (line %d): port '%s' already used above.\n", filename, line, value);
-                               return(-1);
+       if (!!strcmp(value, options.loopback_lcr))
+       {
+               while(searchif) {
+                       ifport = searchif->ifport;
+                       while(ifport) {
+                               if (!strcasecmp(ifport->portname, value)) {
+                                       SPRINT(interface_error, "Error in %s (line %d): port '%s' already used above.\n", filename, line, value);
+                                       return(-1);
+                               }
+                               ifport = ifport->next;
                        }
-//                     /* check for use as GSM */
-//                     if (ifport->gsm) {
-//                             SPRINT(interface_error, "Error in %s (line %d): Interface already used for GSM.\n", filename, line);
-//                             return(-1);
-//                     }
-                       ifport = ifport->next;
+                       searchif = searchif->next;
                }
-               searchif = searchif->next;
        }
        /* alloc port substructure */
        ifport = (struct interface_port *)MALLOC(sizeof(struct interface_port));
@@ -880,24 +885,24 @@ static int inter_tones_dir(struct interface *interface, char *filename, int line
 }
 static int inter_gsm(struct interface *interface, char *filename, int line, char *parameter, char *value)
 {
-#ifndef WITH_GSM
-       SPRINT(interface_error, "Error in %s (line %d): GSM not compiled in.\n", filename, line);
+       SPRINT(interface_error, "Error in %s (line %d): parameter '%s' is outdated.\nPlease use 'gsm-bs' for base station or 'gsm-ms' for mobile station interface!\n", filename, line, parameter);
+       return(-1);
+}
+static int inter_gsm_bs(struct interface *interface, char *filename, int line, char *parameter, char *value)
+{
+#ifndef WITH_GSM_BS
+       SPRINT(interface_error, "Error in %s (line %d): GSM BS side not compiled in.\n", filename, line);
        return(-1);
 #else
        struct interface_port *ifport;
        struct interface *searchif;
 
-       /* check gsm */
-       if (!gsm) {
-               SPRINT(interface_error, "Error in %s (line %d): GSM is not activated.\n", filename, line);
-               return(-1);
-       }
        searchif = interface_newlist;
        while(searchif) {
                ifport = searchif->ifport;
                while(ifport) {
-                       if (ifport->gsm) {
-                               SPRINT(interface_error, "Error in %s (line %d): port '%s' already uses gsm\n", filename, line, value);
+                       if (ifport->gsm_bs) {
+                               SPRINT(interface_error, "Error in %s (line %d): port '%s' already uses gsm BS side.\n", filename, line, ifport->portname);
                                return(-1);
                        }
                        ifport = ifport->next;
@@ -906,13 +911,59 @@ static int inter_gsm(struct interface *interface, char *filename, int line, char
        }
 
        /* set portname */
-       if (inter_portname(interface, filename, line, (char *)"portname", gsm->conf.interface_lcr))
+       if (inter_portname(interface, filename, line, (char *)"portname", options.loopback_lcr))
                return(-1);
-       /* goto end of chain again to set gsmflag*/
+
+       /* goto end of chain again to set gsmflag */
        ifport = interface->ifport;
        while(ifport->next)
                ifport = ifport->next;
-       ifport->gsm = 1;
+       ifport->gsm_bs = 1;
+
+       return(0);
+#endif
+}
+static int inter_gsm_ms(struct interface *interface, char *filename, int line, char *parameter, char *value)
+{
+#ifndef WITH_GSM_MS
+       SPRINT(interface_error, "Error in %s (line %d): GSM MS side not compiled in.\n", filename, line);
+       return(-1);
+#else
+       struct interface_port *ifport, *searchifport;
+       struct interface *searchif;
+
+       /* set portname */
+       if (inter_portname(interface, filename, line, (char *)"portname", options.loopback_lcr))
+               return(-1);
+
+       /* goto end of chain again to set gsmflag and socket */
+       ifport = interface->ifport;
+       while(ifport->next)
+               ifport = ifport->next;
+       ifport->gsm_ms = 1;
+
+       /* copy values */
+       if (!value || !value[0]) {
+               SPRINT(interface_error, "Error in %s (line %d): Missing MS name and socket name.\n", filename, line);
+               return(-1);
+       }
+       SCPY(ifport->gsm_ms_name, value);
+
+       /* check if name is used multiple times */
+       searchif = interface_newlist;
+       while(searchif) {
+               searchifport = searchif->ifport;
+               while(searchifport) {
+                       if (searchifport != ifport 
+                        && !strcmp(searchifport->gsm_ms_name, ifport->gsm_ms_name)) {
+                               SPRINT(interface_error, "Error in %s (line %d): mobile '%s' already uses the given MS name '%s', choose a different one.\n", filename, line, ifport->gsm_ms_name, searchifport->gsm_ms_name);
+                               return(-1);
+                       }
+                       searchifport = searchifport->next;
+               }
+               searchif = searchif->next;
+       }
+
        return(0);
 #endif
 }
@@ -982,6 +1033,42 @@ static int inter_ss5(struct interface *interface, char *filename, int line, char
        return(0);
 }
 #endif
+static int inter_remote(struct interface *interface, char *filename, int line, char *parameter, char *value)
+{
+       struct interface_port *ifport;
+       struct interface *searchif;
+
+       if (!value[0]) {
+               SPRINT(interface_error, "Error in %s (line %d): parameter '%s' expects application name as value.\n", filename, line, parameter);
+               return(-1);
+       }
+       searchif = interface_newlist;
+       while(searchif) {
+               ifport = searchif->ifport;
+               while(ifport) {
+                       if (ifport->remote && !strcmp(ifport->remote_app, value)) {
+                               SPRINT(interface_error, "Error in %s (line %d): port '%s' already uses remote application '%s'.\n", filename, line, ifport->portname, value);
+                               return(-1);
+                       }
+                       ifport = ifport->next;
+               }
+               searchif = searchif->next;
+       }
+
+       /* set portname */
+       if (inter_portname(interface, filename, line, (char *)"portname", options.loopback_lcr))
+               return(-1);
+
+       /* goto end of chain again to set application name */
+       ifport = interface->ifport;
+       while(ifport->next)
+               ifport = ifport->next;
+       ifport->remote = 1;
+       SCPY(ifport->remote_app, value);
+
+
+       return(0);
+}
 
 
 /*
@@ -1055,7 +1142,8 @@ struct interface_param interface_param[] = {
        "Channel selection list for all outgoing calls to the interface.\n"
        "A free channels is searched in order of appearance.\n"
        "This parameter must follow a 'port' parameter.\n"
-       " force - Forces the selected port with no acceptable alternative (see DSS1).\n"
+       " force - Forces the selected port with no acceptable alternative (see Q.931).\n"
+       "  -> this will be automatically set for multipoint (ptmp) NT-mode ports\n"
        " <number>[,...] - List of channels to search.\n"
        " free - Select any free channel\n"
        " any - On outgoing calls, signal 'any channel acceptable'. (see DSS1)\n"
@@ -1111,10 +1199,16 @@ struct interface_param interface_param[] = {
        "To used kernel tones in mISDN_dsp.ko, say 'american', 'german', or 'oldgerman'."},
 
        {"gsm", &inter_gsm, "",
-       "Sets up GSM interface for using OpenBSC.\n"
-       "This interface must be a loopback interface. The second loopback interface\n"
-       "must be assigned to OpenBSC."},
-
+       ""},
+       {"gsm-bs", &inter_gsm_bs, "",
+       "Sets up GSM base station interface for using OpenBSC.\n"
+       "See the default/gsm.conf for configuration for this version."},
+       {"gsm-ms", &inter_gsm_ms, "[<socket>]",
+       "Sets up GSM mobile station interface for using Osmocom-BB.\n"
+       "See the default/gsm.conf for configuration for this version.\n"
+       "The name of the MS folows the interface name.\n"
+       "The socket is /tmp/osmocom_l2 by default and need to be changed when multiple\n"
+       "MS interfaces are used."},
        {"nonotify", &inter_nonotify, "",
        "Prevents sending notify messages to this interface. A call placed on hold will\n"
        "Not affect the remote end (phone or telcom switch).\n"
@@ -1136,6 +1230,10 @@ struct interface_param interface_param[] = {
        " suppress - Suppress received tones, as they will be recognized."},
 #endif
 
+       {"remote", &inter_remote, "<application>",
+       "Sets up an interface that communicates with the remote application.\n"
+       "Use \"asterisk\" to use chan_lcr as remote application."},
+
        {NULL, NULL, NULL, NULL}
 };
 
@@ -1356,6 +1454,9 @@ static void set_defaults(struct interface_port *ifport)
                default_out_channel(ifport);
        if (!ifport->in_channel)
                default_in_channel(ifport);
+       /* must force the channel on PTMP/NT ports */
+       if (!ifport->mISDNport->ptp && ifport->mISDNport->ntmode)
+               ifport->channel_force = 1;
        /* default is_tones */
        if (ifport->interface->is_tones)
                ifport->mISDNport->tones = (ifport->interface->is_tones==IS_YES);
@@ -1388,6 +1489,17 @@ void relink_interfaces(void)
        /* unlink all mISDNports */
        mISDNport = mISDNport_first;
        while(mISDNport) {
+               if (mISDNport->ifport) {
+                       ifport = mISDNport->ifport;
+#ifdef WITH_GSM_MS
+                       if (ifport->gsm_ms)
+                               gsm_ms_delete(ifport->gsm_ms_name);
+#endif
+#ifdef WITH_GSM_BS
+                       if (ifport->gsm_bs)
+                               gsm_bs_exit(0);
+#endif
+               }
                mISDNport->ifport = NULL;
                mISDNport = mISDNport->next;
        }
@@ -1420,7 +1532,7 @@ void relink_interfaces(void)
        while(mISDNport) {
                if (mISDNport->ifport == NULL) {
                        PDEBUG(DEBUG_ISDN, "Port %d is not used anymore and will be closed\n", mISDNport->portnum);
-                       /* remove all port objects and destroy port */
+                       /* destroy port */
                        mISDNport_close(mISDNport);
                        goto closeagain;
                }
@@ -1463,6 +1575,14 @@ void load_port(struct interface_port *ifport)
                set_defaults(ifport);
                /* load static port instances */
                mISDNport_static(mISDNport);
+#ifdef WITH_GSM_MS
+               if (ifport->gsm_ms)
+                       gsm_ms_new(ifport->gsm_ms_name);
+#endif
+#ifdef WITH_GSM_BS
+               if (ifport->gsm_bs)
+                       gsm_bs_init();
+#endif
        } else {
                ifport->block = 2; /* not available */
        }