Review at  https://gerrit.osmocom.org/3768

configure.ac: remove smpp_mirror, which has no relation to a BSC

smpp_mirror is maintained in osmo-msc.git, so let's remove it
(and the associated libsmpp34 build requirement).

Change-Id: Ia7b242cc52cf261b1a5dd3c287da2868e1a2b9a8
---
M configure.ac
M src/utils/Makefile.am
D src/utils/smpp_mirror.c
3 files changed, 0 insertions(+), 387 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/68/3768/1

diff --git a/configure.ac b/configure.ac
index e7f0546..795a307 100644
--- a/configure.ac
+++ b/configure.ac
@@ -50,16 +50,6 @@
 PKG_CHECK_MODULES(LIBOSMOLEGACYMGCP, libosmo-legacy-mgcp >= 0.0.1)
 PKG_CHECK_MODULES(LIBOSMOSCCP, libosmo-sccp >= 0.0.2)
 
-# Enable/disable smpp support in the msc?
-AC_ARG_ENABLE([smpp], [AS_HELP_STRING([--enable-smpp], [Build the SMPP 
interface])],
-    [osmo_ac_build_smpp="$enableval"],[osmo_ac_build_smpp="no"])
-if test "$osmo_ac_build_smpp" = "yes" ; then
-    PKG_CHECK_MODULES(LIBSMPP34, libsmpp34 >= 1.12)
-    AC_DEFINE(BUILD_SMPP, 1, [Define if we want to build SMPP])
-fi
-AM_CONDITIONAL(BUILD_SMPP, test "x$osmo_ac_build_smpp" = "xyes")
-AC_SUBST(osmo_ac_build_smpp)
-
 # Enable/disable transcoding within osmo-bsc_mgcp?
 AC_ARG_ENABLE([mgcp-transcoding], [AS_HELP_STRING([--enable-mgcp-transcoding], 
[Build the MGCP gateway with internal transcoding enabled.])],
     [osmo_ac_mgcp_transcoding="$enableval"],[osmo_ac_mgcp_transcoding="no"])
diff --git a/src/utils/Makefile.am b/src/utils/Makefile.am
index 26494e1..d3f6d62 100644
--- a/src/utils/Makefile.am
+++ b/src/utils/Makefile.am
@@ -11,7 +11,6 @@
        $(LIBOSMOABIS_CFLAGS) \
        $(COVERAGE_CFLAGS) \
        $(SQLITE3_CFLAGS) \
-       $(LIBSMPP34_CFLAGS) \
        $(NULL)
 
 AM_LDFLAGS = \
@@ -43,12 +42,6 @@
        $(NULL)
 endif
 
-if BUILD_SMPP
-noinst_PROGRAMS = \
-       smpp_mirror \
-       $(NULL)
-endif
-
 bs11_config_SOURCES = \
        bs11_config.c \
        $(NULL)
@@ -65,17 +58,6 @@
 
 isdnsync_SOURCES = \
        isdnsync.c \
-       $(NULL)
-
-smpp_mirror_SOURCES = \
-       smpp_mirror.c \
-       $(NULL)
-
-smpp_mirror_LDADD = \
-       $(top_builddir)/src/libcommon/libcommon.a \
-       $(LIBOSMOCORE_LIBS) \
-       $(LIBOSMOGSM_LIBS) \
-       $(LIBSMPP34_LIBS) \
        $(NULL)
 
 meas_vis_SOURCES = \
diff --git a/src/utils/smpp_mirror.c b/src/utils/smpp_mirror.c
deleted file mode 100644
index c570505..0000000
--- a/src/utils/smpp_mirror.c
+++ /dev/null
@@ -1,359 +0,0 @@
-#include <stdio.h>
-#include <unistd.h>
-#include <stdint.h>
-#include <errno.h>
-#include <string.h>
-
-#include <netinet/in.h>
-
-#include <smpp34.h>
-#include <smpp34_structs.h>
-#include <smpp34_params.h>
-
-#include <osmocom/core/application.h>
-#include <osmocom/core/utils.h>
-#include <osmocom/core/msgb.h>
-#include <osmocom/core/talloc.h>
-#include <osmocom/core/select.h>
-#include <osmocom/core/socket.h>
-#include <osmocom/core/write_queue.h>
-
-#include <openbsc/debug.h>
-
-/* FIXME: merge with smpp_smsc.c */
-#define SMPP_SYS_ID_LEN        16
-enum esme_read_state {
-       READ_ST_IN_LEN = 0,
-       READ_ST_IN_MSG = 1,
-};
-/* FIXME: merge with smpp_smsc.c */
-
-struct esme {
-       struct osmo_fd ofd;
-
-       uint32_t own_seq_nr;
-
-       struct osmo_wqueue wqueue;
-       enum esme_read_state read_state;
-       uint32_t read_len;
-       uint32_t read_idx;
-       struct msgb *read_msg;
-
-       uint8_t smpp_version;
-       char system_id[SMPP_SYS_ID_LEN+1];
-       char password[SMPP_SYS_ID_LEN+1];
-};
-
-/* FIXME: merge with smpp_smsc.c */
-#define SMPP34_UNPACK(rc, type, str, data, len)                \
-       memset(str, 0, sizeof(*str));                   \
-       rc = smpp34_unpack(type, str, data, len)
-#define INIT_RESP(type, resp, req)             { \
-       memset((resp), 0, sizeof(*(resp)));       \
-       (resp)->command_length  = 0;              \
-       (resp)->command_id      = type;           \
-       (resp)->command_status  = ESME_ROK;       \
-       (resp)->sequence_number = (req)->sequence_number;       \
-}
-#define PACK_AND_SEND(esme, ptr)       pack_and_send(esme, (ptr)->command_id, 
ptr)
-static inline uint32_t smpp_msgb_cmdid(struct msgb *msg)
-{
-       uint8_t *tmp = msgb_data(msg) + 4;
-       return ntohl(*(uint32_t *)tmp);
-}
-static uint32_t esme_inc_seq_nr(struct esme *esme)
-{
-       esme->own_seq_nr++;
-       if (esme->own_seq_nr > 0x7fffffff)
-               esme->own_seq_nr = 1;
-
-       return esme->own_seq_nr;
-}
-static int pack_and_send(struct esme *esme, uint32_t type, void *ptr)
-{
-       struct msgb *msg = msgb_alloc(4096, "SMPP_Tx");
-       int rc, rlen;
-       if (!msg)
-               return -ENOMEM;
-
-       rc = smpp34_pack(type, msg->tail, msgb_tailroom(msg), &rlen, ptr);
-       if (rc != 0) {
-               LOGP(DSMPP, LOGL_ERROR, "[%s] Error during smpp34_pack(): %s\n",
-                    esme->system_id, smpp34_strerror);
-               msgb_free(msg);
-               return -EINVAL;
-       }
-       msgb_put(msg, rlen);
-
-       if (osmo_wqueue_enqueue(&esme->wqueue, msg) != 0) {
-               LOGP(DSMPP, LOGL_ERROR, "[%s] Write queue full. Dropping 
message\n",
-                    esme->system_id);
-               msgb_free(msg);
-               return -EAGAIN;
-       }
-       return 0;
-}
-/* FIXME: merge with smpp_smsc.c */
-
-static struct tlv_t *find_tlv(struct tlv_t *head, uint16_t tag)
-{
-       struct tlv_t *t;
-
-       for (t = head; t != NULL; t = t->next) {
-               if (t->tag == tag)
-                       return t;
-       }
-       return NULL;
-}
-
-static int smpp_handle_deliver(struct esme *esme, struct msgb *msg)
-{
-       struct deliver_sm_t deliver;
-       struct deliver_sm_resp_t deliver_r;
-       struct submit_sm_t submit;
-       tlv_t *t;
-       int rc;
-
-       memset(&deliver, 0, sizeof(deliver));
-       SMPP34_UNPACK(rc, DELIVER_SM, &deliver, msgb_data(msg), 
msgb_length(msg));
-       if (rc < 0)
-               return rc;
-
-       INIT_RESP(DELIVER_SM_RESP, &deliver_r, &deliver);
-
-       PACK_AND_SEND(esme, &deliver_r);
-
-       memset(&submit, 0, sizeof(submit));
-       submit.command_id = SUBMIT_SM;
-       submit.command_status = ESME_ROK;
-       submit.sequence_number = esme_inc_seq_nr(esme);
-
-       submit.dest_addr_ton =  deliver.source_addr_ton;
-       submit.dest_addr_npi =  deliver.source_addr_npi;
-       memcpy(submit.destination_addr, deliver.source_addr,
-               OSMO_MIN(sizeof(submit.destination_addr),
-                        sizeof(deliver.source_addr)));
-
-       submit.source_addr_ton = deliver.dest_addr_ton;
-       submit.source_addr_npi = deliver.dest_addr_npi;
-       memcpy(submit.source_addr, deliver.destination_addr,
-               OSMO_MIN(sizeof(submit.source_addr),
-                        sizeof(deliver.destination_addr)));
-
-       /* Mirror delivery receipts as a delivery acknowledgements. */
-       if (deliver.esm_class == 0x04) {
-               LOGP(DSMPP, LOGL_DEBUG, "%s\n", deliver.short_message);
-               submit.esm_class = 0x08;
-       } else {
-               submit.esm_class = deliver.esm_class;
-       }
-
-       submit.registered_delivery = deliver.registered_delivery;
-       submit.protocol_id = deliver.protocol_id;
-       submit.priority_flag = deliver.priority_flag;
-       memcpy(submit.schedule_delivery_time, deliver.schedule_delivery_time,
-              OSMO_MIN(sizeof(submit.schedule_delivery_time),
-                       sizeof(deliver.schedule_delivery_time)));
-       memcpy(submit.validity_period, deliver.validity_period,
-               OSMO_MIN(sizeof(submit.validity_period),
-                        sizeof(deliver.validity_period)));
-       submit.registered_delivery = deliver.registered_delivery;
-       submit.replace_if_present_flag = deliver.replace_if_present_flag;
-       submit.data_coding = deliver.data_coding;
-       submit.sm_default_msg_id = deliver.sm_default_msg_id;
-       submit.sm_length = deliver.sm_length;
-       memcpy(submit.short_message, deliver.short_message,
-               OSMO_MIN(sizeof(submit.short_message),
-                        sizeof(deliver.short_message)));
-
-       /* FIXME: More TLV? */
-       t = find_tlv(deliver.tlv, TLVID_user_message_reference);
-       if (t) {
-               tlv_t tlv;
-
-               memset(&tlv, 0, sizeof(tlv));
-               tlv.tag = TLVID_user_message_reference;
-               tlv.length = 2;
-               tlv.value.val16 = t->value.val16;
-               build_tlv(&submit.tlv, &tlv);
-       }
-
-       return PACK_AND_SEND(esme, &submit);
-}
-
-static int bind_transceiver(struct esme *esme)
-{
-       struct bind_transceiver_t bind;
-
-       memset(&bind, 0, sizeof(bind));
-       bind.command_id = BIND_TRANSCEIVER;
-       bind.sequence_number = esme_inc_seq_nr(esme);
-       snprintf((char *)bind.system_id, sizeof(bind.system_id), "%s", 
esme->system_id);
-       snprintf((char *)bind.password, sizeof(bind.password), "%s", 
esme->password);
-       snprintf((char *)bind.system_type, sizeof(bind.system_type), "mirror");
-       bind.interface_version = esme->smpp_version;
-
-       return PACK_AND_SEND(esme, &bind);
-}
-
-static int smpp_pdu_rx(struct esme *esme, struct msgb *msg)
-{
-       uint32_t cmd_id = smpp_msgb_cmdid(msg);
-       int rc;
-
-       switch (cmd_id) {
-       case DELIVER_SM:
-               rc = smpp_handle_deliver(esme, msg);
-               break;
-       default:
-               LOGP(DSMPP, LOGL_NOTICE, "unhandled case %d\n", cmd_id);
-               rc = 0;
-               break;
-       }
-
-       return rc;
-}
-
-/* FIXME: merge with smpp_smsc.c */
-static int esme_read_cb(struct osmo_fd *ofd)
-{
-       struct esme *esme = ofd->data;
-       uint32_t len;
-       uint8_t *lenptr = (uint8_t *) &len;
-       uint8_t *cur;
-       struct msgb *msg;
-       int rdlen;
-       int rc;
-
-       switch (esme->read_state) {
-       case READ_ST_IN_LEN:
-               rdlen = sizeof(uint32_t) - esme->read_idx;
-               rc = read(ofd->fd, lenptr + esme->read_idx, rdlen);
-               if (rc < 0) {
-                       LOGP(DSMPP, LOGL_ERROR, "[%s] read returned %d\n",
-                            esme->system_id, rc);
-               } else if (rc == 0) {
-                       goto dead_socket;
-               } else
-                       esme->read_idx += rc;
-               if (esme->read_idx >= sizeof(uint32_t)) {
-                       esme->read_len = ntohl(len);
-                       msg = msgb_alloc(esme->read_len, "SMPP Rx");
-                       if (!msg)
-                               return -ENOMEM;
-                       esme->read_msg = msg;
-                       cur = msgb_put(msg, sizeof(uint32_t));
-                       memcpy(cur, lenptr, sizeof(uint32_t));
-                       esme->read_state = READ_ST_IN_MSG;
-                       esme->read_idx = sizeof(uint32_t);
-               }
-               break;
-       case READ_ST_IN_MSG:
-               msg = esme->read_msg;
-               rdlen = esme->read_len - esme->read_idx;
-               rc = read(ofd->fd, msg->tail, OSMO_MIN(rdlen, 
msgb_tailroom(msg)));
-               if (rc < 0) {
-                       LOGP(DSMPP, LOGL_ERROR, "[%s] read returned %d\n",
-                               esme->system_id, rc);
-               } else if (rc == 0) {
-                       goto dead_socket;
-               } else {
-                       esme->read_idx += rc;
-                       msgb_put(msg, rc);
-               }
-
-               if (esme->read_idx >= esme->read_len) {
-                       rc = smpp_pdu_rx(esme, esme->read_msg);
-                       esme->read_msg = NULL;
-                       esme->read_idx = 0;
-                       esme->read_len = 0;
-                       esme->read_state = READ_ST_IN_LEN;
-               }
-               break;
-       }
-
-       return 0;
-dead_socket:
-       msgb_free(esme->read_msg);
-       osmo_fd_unregister(&esme->wqueue.bfd);
-       close(esme->wqueue.bfd.fd);
-       esme->wqueue.bfd.fd = -1;
-       exit(2342);
-
-       return 0;
-}
-
-static int esme_write_cb(struct osmo_fd *ofd, struct msgb *msg)
-{
-       struct esme *esme = ofd->data;
-       int rc;
-
-       rc = write(ofd->fd, msgb_data(msg), msgb_length(msg));
-       if (rc == 0) {
-               osmo_fd_unregister(&esme->wqueue.bfd);
-               close(esme->wqueue.bfd.fd);
-               esme->wqueue.bfd.fd = -1;
-               exit(99);
-       } else if (rc < msgb_length(msg)) {
-               LOGP(DSMPP, LOGL_ERROR, "[%s] Short write\n", esme->system_id);
-               return 0;
-       }
-
-       return 0;
-}
-
-static int smpp_esme_init(struct esme *esme, const char *host, uint16_t port)
-{
-       int rc;
-
-       if (port == 0)
-               port = 2775;
-
-       esme->own_seq_nr = rand();
-       esme_inc_seq_nr(esme);
-       osmo_wqueue_init(&esme->wqueue, 10);
-       esme->wqueue.bfd.data = esme;
-       esme->wqueue.read_cb = esme_read_cb;
-       esme->wqueue.write_cb = esme_write_cb;
-
-       rc = osmo_sock_init_ofd(&esme->wqueue.bfd, AF_UNSPEC, SOCK_STREAM,
-                               IPPROTO_TCP, host, port, OSMO_SOCK_F_CONNECT);
-       if (rc < 0)
-               return rc;
-
-       return bind_transceiver(esme);
-}
-
-
-int main(int argc, char **argv)
-{
-       struct esme esme;
-       char *host = "localhost";
-       int port = 0;
-       int rc;
-
-       msgb_talloc_ctx_init(NULL, 0);
-
-       memset(&esme, 0, sizeof(esme));
-
-       osmo_init_logging(&log_info);
-
-       snprintf((char *) esme.system_id, sizeof(esme.system_id), "mirror");
-       snprintf((char *) esme.password, sizeof(esme.password), "mirror");
-       esme.smpp_version = 0x34;
-
-       if (argc >= 2)
-               host = argv[1];
-       if (argc >= 3)
-               port = atoi(argv[2]);
-
-       rc = smpp_esme_init(&esme, host, port);
-       if (rc < 0)
-               exit(1);
-
-       while (1) {
-               osmo_select_main(0);
-       }
-
-       exit(0);
-}

-- 
To view, visit https://gerrit.osmocom.org/3768
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia7b242cc52cf261b1a5dd3c287da2868e1a2b9a8
Gerrit-PatchSet: 1
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <[email protected]>

Reply via email to