Move TRP messaging to tr_msg.c. Fix old bug.
[trust_router.git] / common / tr_msg.c
index cb60292..0d55ebe 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2012, JANET(UK)
+ * Copyright (c) 2012-2014 , JANET(UK)
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  */
-
+#include <sys/socket.h>
+#include <netinet/in.h>
+#include <arpa/inet.h>
 #include <string.h>
 #include <openssl/dh.h>
 #include <jansson.h>
+#include <assert.h>
+#include <talloc.h>
+
 
 #include <tr_msg.h>
-#include <tid.h>
+#include <trust_router/tr_name.h>
+#include <tid_internal.h>
+#include <trp_internal.h>
+#include <trust_router/tr_constraint.h>
+#include <tr_debug.h>
+
+/* JSON helpers */
+/* Read attribute attr from msg as an integer. Returns nonzero on error. */
+static int tr_msg_get_json_integer(json_t *jmsg, const char *attr, int *dest)
+{
+  json_t *obj;
+
+  obj=json_object_get(jmsg, attr);
+  if (obj == NULL) {
+    return -1;
+  }
+  /* check type */
+  if (!json_is_integer(obj)) {
+    return -1;
+  }
+
+  (*dest)=json_integer_value(obj);
+  return 0;
+}
+
+/* Read attribute attr from msg as a string. Copies string into mem_ctx context so jmsg can
+ * be destroyed safely. Returns nonzero on error. */
+static int tr_msg_get_json_string(json_t *jmsg, const char *attr, char **dest, TALLOC_CTX *mem_ctx)
+{
+  json_t *obj;
+
+  obj=json_object_get(jmsg, attr);
+  if (obj == NULL)
+    return -1;
+
+  /* check type */
+  if (!json_is_string(obj))
+    return -1;
+
+  *dest=talloc_strdup(mem_ctx, json_string_value(obj));
+  if (*dest==NULL)
+    return -1;
+
+  return 0;
+}
+
+enum msg_type tr_msg_get_msg_type(TR_MSG *msg) 
+{
+  return msg->msg_type;
+}
+
+void tr_msg_set_msg_type(TR_MSG *msg, enum msg_type type)
+{
+  msg->msg_type = type;
+}
+
+TID_REQ *tr_msg_get_req(TR_MSG *msg)
+{
+  if (msg->msg_type == TID_REQUEST)
+    return (TID_REQ *)msg->msg_rep;
+  return NULL;
+}
+
+void tr_msg_set_req(TR_MSG *msg, TID_REQ *req)
+{
+  msg->msg_rep = req;
+  msg->msg_type = TID_REQUEST;
+}
+
+TID_RESP *tr_msg_get_resp(TR_MSG *msg)
+{
+  if (msg->msg_type == TID_RESPONSE)
+    return (TID_RESP *)msg->msg_rep;
+  return NULL;
+}
+
+void tr_msg_set_resp(TR_MSG *msg, TID_RESP *resp)
+{
+  msg->msg_rep = resp;
+  msg->msg_type = TID_RESPONSE;
+}
+
+TRP_UPD *tr_msg_get_trp_upd(TR_MSG *msg)
+{
+  if (msg->msg_type == TRP_UPDATE)
+    return (TRP_UPD *)msg->msg_rep;
+  return NULL;
+}
+
+void tr_msg_set_trp_upd(TR_MSG *msg, TRP_UPD *update)
+{
+  msg->msg_rep=update;
+  msg->msg_type=TRP_UPDATE;
+}
+
+TRP_REQ *tr_msg_get_trp_req(TR_MSG *msg)
+{
+  if (msg->msg_type == TRP_REQUEST)
+    return (TRP_REQ *)msg->msg_rep;
+  return NULL;
+}
+
+void tr_msg_set_trp_req(TR_MSG *msg, TRP_REQ *req)
+{
+  msg->msg_rep=req;
+  msg->msg_type=TRP_REQUEST;
+}
 
 static json_t *tr_msg_encode_dh(DH *dh)
 {
@@ -64,40 +175,34 @@ static json_t *tr_msg_encode_dh(DH *dh)
 static DH *tr_msg_decode_dh(json_t *jdh)
 {
   DH *dh = NULL;
-  json_error_t rc;
   json_t *jp = NULL;
   json_t *jg = NULL;
   json_t *jpub_key = NULL;
-  int msize;
 
   if (!(dh = malloc(sizeof(DH)))) {
-    fprintf (stderr, "tr_msg_decode_dh(): Error allocating DH structure.\n");
+    tr_crit("tr_msg_decode_dh(): Error allocating DH structure.");
     return NULL;
   }
  
   memset(dh, 0, sizeof(DH));
 
   /* store required fields from dh object */
-  if (((msize = json_object_size(jdh)) < 3) ||
-      (NULL == (jp = json_object_get(jdh, "dh_p"))) ||
-      (!json_is_string(jp)) ||
+  if ((NULL == (jp = json_object_get(jdh, "dh_p"))) ||
       (NULL == (jg = json_object_get(jdh, "dh_g"))) ||
-      (!json_is_string(jg)) ||
-      (NULL == (jpub_key = json_object_get(jdh, "dh_pub_key"))) ||
-      (!json_is_string(jdh))) {
-    fprintf (stderr, "tr_msg_decode(): Error parsing message.\n");
+      (NULL == (jpub_key = json_object_get(jdh, "dh_pub_key")))) {
+    tr_debug("tr_msg_decode_dh(): Error parsing dh_info.");
     free(dh);
     return NULL;
   }
 
-  dh->p = json_string_value(jp);
-  dh->g = json_string_value(jg);
-  dh->pub_key = json_string_value(jpub_key);
+  BN_hex2bn(&(dh->p), json_string_value(jp));
+  BN_hex2bn(&(dh->g), json_string_value(jg));
+  BN_hex2bn(&(dh->pub_key), json_string_value(jpub_key));
 
   return dh;
 }
 
-json_t *tr_msg_encode_tidreq(TID_REQ *req)
+static json_t * tr_msg_encode_tidreq(TID_REQ *req)
 {
   json_t *jreq = NULL;
   json_t *jstr = NULL;
@@ -105,7 +210,7 @@ json_t *tr_msg_encode_tidreq(TID_REQ *req)
   if ((!req) || (!req->rp_realm) || (!req->realm) || !(req->comm))
     return NULL;
 
-  jreq = json_object();
+  assert(jreq = json_object());
 
   jstr = json_string(req->rp_realm->buf);
   json_object_set_new(jreq, "rp_realm", jstr);
@@ -115,75 +220,224 @@ json_t *tr_msg_encode_tidreq(TID_REQ *req)
 
   jstr = json_string(req->comm->buf);
   json_object_set_new(jreq, "community", jstr);
+  
+  if (req->orig_coi) {
+    jstr = json_string(req->orig_coi->buf);
+    json_object_set_new(jreq, "orig_coi", jstr);
+  }
 
   json_object_set_new(jreq, "dh_info", tr_msg_encode_dh(req->tidc_dh));
+
+  if (req->cons)
+    json_object_set(jreq, "constraints", (json_t *) req->cons);
+
+  if (req->path)
+    json_object_set(jreq, "path", req->path);
+  if (req->expiration_interval)
+    json_object_set_new(jreq, "expiration_interval",
+                       json_integer(req->expiration_interval));
   
   return jreq;
 }
 
-TID_REQ *tr_msg_decode_tidreq(json_t *jreq)
+static TID_REQ *tr_msg_decode_tidreq(json_t *jreq)
 {
   TID_REQ *treq = NULL;
-  json_error_t rc;
   json_t *jrp_realm = NULL;
   json_t *jrealm = NULL;
   json_t *jcomm = NULL;
   json_t *jorig_coi = NULL;
   json_t *jdh = NULL;
-  int msize;
+  json_t *jpath = NULL;
+  json_t *jexpire_interval = NULL;
 
-  if (!(treq = malloc(sizeof(TID_REQ)))) {
-    fprintf (stderr, "tr_msg_decode_tidreq(): Error allocating TID_REQ structure.\n");
+  if (!(treq =tid_req_new())) {
+    tr_crit("tr_msg_decode_tidreq(): Error allocating TID_REQ structure.");
     return NULL;
   }
  
-  memset(treq, 0, sizeof(TID_REQ));
-
   /* store required fields from request */
-  if (((msize = json_object_size(jreq)) < 4) ||
-      (NULL == (jrp_realm = json_object_get(jreq, "rp_realm"))) ||
-      (!json_is_string(jrp_realm)) ||
-      (NULL == (jrealm = json_object_get(jreq, "realm"))) ||
-      (!json_is_string(jrealm)) ||
-      (NULL == (jcomm = json_object_get(jreq, "comm"))) ||
-      (!json_is_string(jcomm)) ||
-      (NULL == (jdh = json_object_get(jreq, "dh_info"))) ||
-      (!json_is_object(jdh))) {
-    fprintf (stderr, "tr_msg_decode(): Error parsing message.\n");
-    free(treq);
+  if ((NULL == (jrp_realm = json_object_get(jreq, "rp_realm"))) ||
+      (NULL == (jrealm = json_object_get(jreq, "target_realm"))) ||
+      (NULL == (jcomm = json_object_get(jreq, "community")))) {
+    tr_notice("tr_msg_decode(): Error parsing required fields.");
+    tid_req_free(treq);
     return NULL;
   }
 
-  treq->rp_realm->buf = json_string_value(jrp_realm);
-  treq->rp_realm->len = strlen(treq->rp_realm->buf);
-  treq->realm->buf = json_string_value(jrealm);
-  treq->realm->len = strlen(treq->realm->buf);
-  treq->comm->buf = json_string_value(jcomm);
-  treq->comm->len = strlen(treq->comm->buf);
+  jpath = json_object_get(jreq, "path");
+  jexpire_interval = json_object_get(jreq, "expiration_interval");
+
+  treq->rp_realm = tr_new_name((char *)json_string_value(jrp_realm));
+  treq->realm = tr_new_name((char *)json_string_value(jrealm));
+  treq->comm = tr_new_name((char *)json_string_value(jcomm));
+
+  /* Get DH Info from the request */
+  if (NULL == (jdh = json_object_get(jreq, "dh_info"))) {
+    tr_debug("tr_msg_decode(): Error parsing dh_info.");
+    tid_req_free(treq);
+    return NULL;
+  }
   treq->tidc_dh = tr_msg_decode_dh(jdh);
 
   /* store optional "orig_coi" field */
-  if ((NULL != (jorig_coi = json_object_get(jreq, "orig_coi"))) &&
-      (!json_is_object(jorig_coi))) {
-    treq->orig_coi->buf = json_string_value(jorig_coi);
-    treq->orig_coi->len = strlen(treq->orig_coi->buf);
+  if (NULL != (jorig_coi = json_object_get(jreq, "orig_coi"))) {
+    treq->orig_coi = tr_new_name((char *)json_string_value(jorig_coi));
   }
 
+  treq->cons = (TR_CONSTRAINT_SET *) json_object_get(jreq, "constraints");
+  if (treq->cons) {
+    if (!tr_constraint_set_validate(treq->cons)) {
+      tr_debug("Constraint set validation failed");
+    tid_req_free(treq);
+    return NULL;
+    }
+    json_incref((json_t *) treq->cons);
+    tid_req_cleanup_json(treq, (json_t *) treq->cons);
+  }
+  if (jpath) {
+    json_incref(jpath);
+    treq->path = jpath;
+    tid_req_cleanup_json(treq, jpath);
+  }
+  if (jexpire_interval)
+    treq->expiration_interval = json_integer_value(jexpire_interval);
+  
   return treq;
 }
 
-json_t *tr_msg_encode_tidresp(TID_RESP *resp)
+static json_t *tr_msg_encode_one_server(TID_SRVR_BLK *srvr)
+{
+  json_t *jsrvr = NULL;
+  json_t *jstr = NULL;
+  gchar *time_str = g_time_val_to_iso8601(&srvr->key_expiration);
+
+  tr_debug("Encoding one server.");
+
+  jsrvr = json_object();
+
+  /* Server IP Address -- TBD handle IPv6 */
+  jstr = json_string(inet_ntoa(srvr->aaa_server_addr));
+  json_object_set_new(jsrvr, "server_addr", jstr);
+
+  json_object_set_new(jsrvr,
+                     "key_expiration", json_string(time_str));
+  g_free(time_str);
+  /* Server DH Block */
+  jstr = json_string(srvr->key_name->buf);
+  json_object_set_new(jsrvr, "key_name", jstr);
+  json_object_set_new(jsrvr, "server_dh", tr_msg_encode_dh(srvr->aaa_server_dh));
+  if (srvr->path)
+    /* The path is owned by the srvr, so grab an extra ref*/
+    json_object_set(jsrvr, "path", srvr->path);
+  return jsrvr;
+}
+
+static int tr_msg_decode_one_server(json_t *jsrvr, TID_SRVR_BLK *srvr) 
+{
+  json_t *jsrvr_addr = NULL;
+  json_t *jsrvr_kn = NULL;
+  json_t *jsrvr_dh = NULL;
+  json_t *jsrvr_expire = NULL;
+
+  if (jsrvr == NULL)
+    return -1;
+
+
+  if ((NULL == (jsrvr_addr = json_object_get(jsrvr, "server_addr"))) ||
+      (NULL == (jsrvr_kn = json_object_get(jsrvr, "key_name"))) ||
+      (NULL == (jsrvr_dh = json_object_get(jsrvr, "server_dh")))) {
+    tr_notice("tr_msg_decode_one_server(): Error parsing required fields.");
+    return -1;
+  }
+  
+  /* TBD -- handle IPv6 Addresses */
+  inet_aton(json_string_value(jsrvr_addr), &(srvr->aaa_server_addr));
+  srvr->key_name = tr_new_name((char *)json_string_value(jsrvr_kn));
+  srvr->aaa_server_dh = tr_msg_decode_dh(jsrvr_dh);
+  srvr->path = json_object_get(jsrvr, "path");
+  jsrvr_expire = json_object_get(jsrvr, "key_expiration");
+  if (jsrvr_expire && json_is_string(jsrvr_expire)) {
+    if (!g_time_val_from_iso8601(json_string_value(jsrvr_expire),
+                                &srvr->key_expiration))
+      tr_notice("Key expiration %s cannot be parsed", json_string_value(jsrvr_expire));
+  }
+  
+  return 0;
+}
+
+static json_t *tr_msg_encode_servers(TID_RESP *resp)
+{
+  json_t *jservers = NULL;
+  json_t *jsrvr = NULL;
+  TID_SRVR_BLK *srvr = NULL;
+  size_t index;
+
+  jservers = json_array();
+
+  tid_resp_servers_foreach(resp, srvr, index) {
+    if ((NULL == (jsrvr = tr_msg_encode_one_server(srvr))) ||
+       (-1 == json_array_append_new(jservers, jsrvr))) {
+      return NULL;
+    }
+  }
+
+  //  tr_debug("tr_msg_encode_servers(): servers contains:");
+  //  tr_debug("%s", json_dumps(jservers, 0));
+  return jservers;
+}
+
+static TID_SRVR_BLK *tr_msg_decode_servers(void * ctx, json_t *jservers, size_t *out_len)
+{
+  TID_SRVR_BLK *servers = NULL;
+  json_t *jsrvr;
+  size_t i, num_servers;
+
+  num_servers = json_array_size(jservers);
+  tr_debug("tr_msg_decode_servers(): Number of servers = %u.", (unsigned) num_servers);
+  
+  if (0 == num_servers) {
+    tr_debug("tr_msg_decode_servers(): Server array is empty."); 
+    return NULL;
+  }
+  servers = talloc_zero_array(ctx, TID_SRVR_BLK, num_servers);
+
+  for (i = 0; i < num_servers; i++) {
+    jsrvr = json_array_get(jservers, i);
+    if (0 != tr_msg_decode_one_server(jsrvr, &servers[i])) {
+      talloc_free(servers);
+      return NULL;
+    }
+
+
+  }
+  *out_len = num_servers;
+  return servers;
+}
+
+static json_t * tr_msg_encode_tidresp(TID_RESP *resp)
 {
   json_t *jresp = NULL;
   json_t *jstr = NULL;
+  json_t *jservers = NULL;
 
-  if ((!resp) || (!resp->result) || (!resp->rp_realm) || (!resp->realm) || !(resp->comm))
+  if ((!resp) || (!resp->rp_realm) || (!resp->realm) || !(resp->comm))
     return NULL;
 
   jresp = json_object();
 
-  jstr = json_string(resp->result->buf);
-  json_object_set_new(jresp, "result", jstr);
+  if (TID_ERROR == resp->result) {
+    jstr = json_string("error");
+    json_object_set_new(jresp, "result", jstr);
+    if (resp->err_msg) {
+      jstr = json_string(resp->err_msg->buf);
+      json_object_set_new(jresp, "err_msg", jstr);
+    }
+  }
+  else {
+    jstr = json_string("success");
+    json_object_set_new(jresp, "result", jstr);
+  }
 
   jstr = json_string(resp->rp_realm->buf);
   json_object_set_new(jresp, "rp_realm", jstr);
@@ -199,73 +453,415 @@ json_t *tr_msg_encode_tidresp(TID_RESP *resp)
     json_object_set_new(jresp, "orig_coi", jstr);
   }
 
-  // TBD -- Encode server info.
+  if (NULL == resp->servers) {
+    tr_debug("tr_msg_encode_tidresp(): No servers to encode.");
+  }
+  else {
+    jservers = tr_msg_encode_servers(resp);
+    json_object_set_new(jresp, "servers", jservers);
+  }
+  if (resp->error_path)
+    json_object_set(jresp, "error_path", resp->error_path);
+  
   
   return jresp;
 }
 
-TID_RESP *tr_msg_decode_tidresp(json_t *jresp)
+static TID_RESP *tr_msg_decode_tidresp(json_t *jresp)
 {
   TID_RESP *tresp = NULL;
-  json_error_t rc;
   json_t *jresult = NULL;
   json_t *jrp_realm = NULL;
   json_t *jrealm = NULL;
   json_t *jcomm = NULL;
   json_t *jorig_coi = NULL;
   json_t *jservers = NULL;
-  int msize;
+  json_t *jerr_msg = NULL;
 
-  if (!(tresp = malloc(sizeof(TID_RESP)))) {
-    fprintf (stderr, "tr_msg_decode_tidresp(): Error allocating TID_RESP structure.\n");
+  if (!(tresp=tid_resp_new(NULL))) {
+    tr_crit("tr_msg_decode_tidresp(): Error allocating TID_RESP structure.");
     return NULL;
   }
  
-  memset(tresp, 0, sizeof(TID_RESP));
 
-  /* store required fields from request */
-  if (((msize = json_object_size(jresp)) < 5) ||
-      (NULL == (jresult = json_object_get(jresp, "result"))) ||
+  /* store required fields from response */
+  if ((NULL == (jresult = json_object_get(jresp, "result"))) ||
       (!json_is_string(jresult)) ||
       (NULL == (jrp_realm = json_object_get(jresp, "rp_realm"))) ||
       (!json_is_string(jrp_realm)) ||
-      (NULL == (jrealm = json_object_get(jresp, "realm"))) ||
+      (NULL == (jrealm = json_object_get(jresp, "target_realm"))) ||
       (!json_is_string(jrealm)) ||
       (NULL == (jcomm = json_object_get(jresp, "comm"))) ||
-      (!json_is_string(jcomm)) ||
-      (NULL == (jservers = json_object_get(jresp, "servers"))) ||
-      (!json_is_object(jservers))) {
-    fprintf (stderr, "tr_msg_decode(): Error parsing message.\n");
-    free(tresp);
+      (!json_is_string(jcomm))) {
+    tr_debug("tr_msg_decode_tidresp(): Error parsing response.");
+    talloc_free(tresp);
     return NULL;
   }
 
-  tresp->result->buf = json_string_value(jresult);
-  tresp->result->len = strlen(tresp->result->buf);
-  tresp->rp_realm->buf = json_string_value(jrp_realm);
-  tresp->rp_realm->len = strlen(tresp->rp_realm->buf);
-  tresp->realm->buf = json_string_value(jrealm);
-  tresp->realm->len = strlen(tresp->realm->buf);
-  tresp->comm->buf = json_string_value(jcomm);
-  tresp->comm->len = strlen(tresp->comm->buf);
+  if (0 == (strcmp(json_string_value(jresult), "success"))) {
+    tr_debug("tr_msg_decode_tidresp(): Success! result = %s.", json_string_value(jresult));
+    if ((NULL != (jservers = json_object_get(jresp, "servers"))) ||
+       (!json_is_array(jservers))) {
+      tresp->servers = tr_msg_decode_servers(tresp, jservers, &tresp->num_servers); 
+    } 
+    else {
+      talloc_free(tresp);
+      return NULL;
+    }
+    tresp->result = TID_SUCCESS;
+  }
+  else {
+    tresp->result = TID_ERROR;
+    tr_debug("tr_msg_decode_tidresp(): Error! result = %s.", json_string_value(jresult));
+    if ((NULL != (jerr_msg = json_object_get(jresp, "err_msg"))) ||
+       (!json_is_string(jerr_msg))) {
+      tresp->err_msg = tr_new_name((char *)json_string_value(jerr_msg));
+    }
+  }
+
+  tresp->rp_realm = tr_new_name((char *)json_string_value(jrp_realm));
+  tresp->realm = tr_new_name((char *)json_string_value(jrealm));
+  tresp->comm = tr_new_name((char *)json_string_value(jcomm));
 
   /* store optional "orig_coi" field */
   if ((NULL != (jorig_coi = json_object_get(jresp, "orig_coi"))) &&
       (!json_is_object(jorig_coi))) {
-    tresp->orig_coi->buf = json_string_value(jorig_coi);
-    tresp->orig_coi->len = strlen(tresp->orig_coi->buf);
+    tresp->orig_coi = tr_new_name((char *)json_string_value(jorig_coi));
+  }
+     
+  return tresp;
+}
+
+
+/* Information records for TRP update msg 
+ * requires that jrec already be allocated */
+static TRP_RC tr_msg_encode_inforec_route(json_t *jrec, TRP_INFOREC_DATA rec_data)
+{
+  TRP_INFOREC_ROUTE *route=rec_data.route;
+  json_t *jstr=NULL;
+  json_t *jint=NULL;
+  char *s=NULL;
+
+  if (route==NULL)
+    return TRP_BADTYPE;
+
+  s=tr_name_strdup(route->comm);
+  if (s==NULL)
+    return TRP_NOMEM;
+  jstr=json_string(s);
+  free(s);s=NULL;
+  if(jstr==NULL)
+    return TRP_ERROR;
+  json_object_set_new(jrec, "community", jstr);
+
+  s=tr_name_strdup(route->realm);
+  if (s==NULL)
+    return TRP_NOMEM;
+  jstr=json_string(s);
+  free(s);s=NULL;
+  if(jstr==NULL)
+    return TRP_ERROR;
+  json_object_set_new(jrec, "realm", jstr);
+
+  s=tr_name_strdup(route->trust_router);
+  if (s==NULL)
+    return TRP_NOMEM;
+  jstr=json_string(s);
+  free(s);s=NULL;
+  if(jstr==NULL)
+    return TRP_ERROR;
+  json_object_set_new(jrec, "trust_router", jstr);
+
+  jint=json_integer(route->metric);
+  if(jint==NULL)
+    return TRP_ERROR;
+  json_object_set_new(jrec, "metric", jint);
+
+  jint=json_integer(route->interval);
+  if(jint==NULL)
+    return TRP_ERROR;
+  json_object_set_new(jrec, "interval", jint);
+
+  return TRP_SUCCESS;
+}
+
+static json_t *tr_msg_encode_inforec(TRP_INFOREC *rec)
+{
+  json_t *jrec=NULL;
+  json_t *jstr=NULL;
+
+  if ((rec==NULL) || (rec->type==TRP_INFOREC_TYPE_UNKNOWN))
+    return NULL;
+
+  jrec=json_object();
+  if (jrec==NULL)
+    return NULL;
+
+  jstr=json_string(trp_inforec_type_to_string(rec->type));
+  if (jstr==NULL) {
+    json_decref(jrec);
+    return NULL;
   }
+  json_object_set_new(jrec, "record_type", jstr);
+
+  switch (rec->type) {
+  case TRP_INFOREC_TYPE_ROUTE:
+    if (TRP_SUCCESS!=tr_msg_encode_inforec_route(jrec, rec->data)) {
+      json_decref(jrec);
+      return NULL;
+    }
+    break;
+  default:
+    json_decref(jrec);
+    return NULL;
+  }
+  return jrec;
+}
 
-  //  Decode server info
-  //  tresp->servers = tr_msg_decode_servers(jservers); 
+/* decode a single record */
+static TRP_INFOREC *tr_msg_decode_trp_inforec(TALLOC_CTX *mem_ctx, json_t *jrecord)
+{
+  TALLOC_CTX *tmp_ctx=talloc_new(NULL);
+  TRP_INFOREC_TYPE rectype;
+  TRP_INFOREC *rec=NULL;
+  TRP_RC rc=TRP_ERROR;
+  char *s=NULL;
+  int num=0;
   
-  return tresp;
+  if (0!=tr_msg_get_json_string(jrecord, "record_type", &s, tmp_ctx))
+    goto cleanup;
+
+  rectype=trp_inforec_type_from_string(s);
+  talloc_free(s); s=NULL;
+
+  rec=trp_inforec_new(tmp_ctx, rectype);
+  if (rec==NULL) {
+    rc=TRP_NOMEM;
+    goto cleanup;
+  }
+
+  /* We only support route_info records for now*/
+  if (rec->type!=TRP_INFOREC_TYPE_ROUTE) {
+    rc=TRP_UNSUPPORTED;
+    goto cleanup;
+  }
+
+  tr_debug("tr_msg_decode_trp_inforec: '%s' record found.", trp_inforec_type_to_string(rec->type));
+
+  rc=tr_msg_get_json_string(jrecord, "community", &s, tmp_ctx);
+  if (rc != TRP_SUCCESS)
+    goto cleanup;
+  if (TRP_SUCCESS!=trp_inforec_set_comm(rec, tr_new_name(s)))
+    goto cleanup;
+  talloc_free(s); s=NULL;
+
+  rc=tr_msg_get_json_string(jrecord, "realm", &s, tmp_ctx);
+  if (rc != TRP_SUCCESS)
+    goto cleanup;
+  if (TRP_SUCCESS!=trp_inforec_set_realm(rec, tr_new_name(s))) /* assumes route_info */
+    goto cleanup;
+  talloc_free(s); s=NULL;
+
+  rc=tr_msg_get_json_string(jrecord, "trust_router", &s, tmp_ctx);
+  if (rc != TRP_SUCCESS)
+    goto cleanup;
+  if (TRP_SUCCESS!=trp_inforec_set_trust_router(rec, tr_new_name(s))) /* assumes route_info */
+    goto cleanup;
+  talloc_free(s); s=NULL;
+
+  rc=tr_msg_get_json_integer(jrecord, "metric", &num);
+  if ((rc != TRP_SUCCESS) || (TRP_SUCCESS!=trp_inforec_set_metric(rec,num)))
+    goto cleanup;
+
+  rc=tr_msg_get_json_integer(jrecord, "interval", &num);
+  if ((rc != TRP_SUCCESS) || (TRP_SUCCESS!=trp_inforec_set_interval(rec,num)))
+    goto cleanup;
+
+  talloc_steal(mem_ctx, rec);
+  rc=TRP_SUCCESS;
+
+cleanup:
+  if (rc != TRP_SUCCESS) {
+    trp_inforec_free(rec);
+    rec=NULL;
+  }
+  talloc_free(tmp_ctx);
+  return rec;
+}
+
+/* TRP update msg */
+static json_t *tr_msg_encode_trp_upd(TRP_UPD *update)
+{
+  json_t *jupdate=NULL;
+  json_t *jrecords=NULL;
+  json_t *jrec=NULL;
+  TRP_INFOREC *rec;
+
+  if (update==NULL)
+    return NULL;
+
+  jupdate=json_object();
+  if (jupdate==NULL)
+    return NULL;
+
+  jrecords=json_array();
+  if (jrecords==NULL) {
+    json_decref(jupdate);
+    return NULL;
+  }
+  json_object_set_new(jupdate, "records", jrecords); /* jrecords now a "borrowed" reference */
+  for (rec=update->records; rec!=NULL; rec=rec->next) {
+    jrec=tr_msg_encode_inforec(rec);
+    if (jrec==NULL) {
+      json_decref(jupdate); /* also decs jrecords and any elements */
+      return NULL;
+    }
+    if (0!=json_array_append_new(jrecords, jrec)) {
+      json_decref(jupdate); /* also decs jrecords and any elements */
+      json_decref(jrec); /* this one did not get added so dec explicitly */
+      return NULL;
+    }
+  }
+
+  return jupdate;
+}
+
+/*Creates a linked list of records in the msg->body talloc context.
+ * An error will be returned if any unparseable records are encountered. 
+ */
+static TRP_UPD *tr_msg_decode_trp_upd(TALLOC_CTX *mem_ctx, json_t *jupdate)
+{
+  TALLOC_CTX *tmp_ctx=talloc_new(NULL);
+  json_t *jrecords=NULL;
+  size_t ii=0;
+  TRP_UPD *update=NULL;
+  TRP_INFOREC *new_rec=NULL;
+  TRP_INFOREC *list_tail=NULL;
+  TRP_RC rc=TRP_ERROR;
+
+  update=trp_upd_new(tmp_ctx);
+  if (update==NULL) {
+    rc=TRP_NOMEM;
+    goto cleanup;
+  }
+
+  jrecords=json_object_get(jupdate, "records");
+  if ((jrecords==NULL) || (!json_is_array(jrecords))) {
+    rc=TRP_NOPARSE;
+    goto cleanup;
+  }
+
+  tr_debug("tr_msg_decode_trp_upd: found %d records", json_array_size(jrecords));
+  /* process the array */
+  for (ii=0; ii<json_array_size(jrecords); ii++) {
+    new_rec=tr_msg_decode_trp_inforec(update, json_array_get(jrecords, ii));
+    if (new_rec==NULL) {
+      rc=TRP_NOPARSE;
+      goto cleanup;
+    }
+
+    if (list_tail==NULL)
+      update->records=new_rec; /* first is a special case */
+    else
+      list_tail->next=new_rec;
+
+    list_tail=new_rec;
+  }
+
+  /* Succeeded. Move new allocations into the correct talloc context */
+  talloc_steal(mem_ctx, update);
+  rc=TRP_SUCCESS;
+
+cleanup:
+  talloc_free(tmp_ctx);
+  if (rc!=TRP_SUCCESS)
+    return NULL;
+  return update;
+}
+
+static json_t *tr_msg_encode_trp_req(TRP_REQ *req)
+{
+  json_t *jbody=NULL;
+  json_t *jstr=NULL;
+  char *s=NULL;
+
+  if (req==NULL)
+    return NULL;
+
+  jbody=json_object();
+  if (jbody==NULL)
+    return NULL;
+
+  s=tr_name_strdup(req->comm); /* ensures null termination */
+  if (s==NULL) {
+    json_decref(jbody);
+    return NULL;
+  }
+  jstr=json_string(s);
+  free(s); s=NULL;
+  if (jstr==NULL) {
+    json_decref(jbody);
+    return NULL;
+  }
+  json_object_set_new(jbody, "community", jstr);
+    
+  s=tr_name_strdup(req->realm); /* ensures null termination */
+  if (s==NULL) {
+    json_decref(jbody);
+    return NULL;
+  }
+  jstr=json_string(s);
+  free(s); s=NULL;
+  if (jstr==NULL) {
+    json_decref(jbody);
+    return NULL;
+  }
+  json_object_set_new(jbody, "realm", jstr);
+
+  return jbody;
+}
+
+static TRP_REQ *tr_msg_decode_trp_req(TALLOC_CTX *mem_ctx, json_t *jreq)
+{
+  TALLOC_CTX *tmp_ctx=talloc_new(NULL);
+  TRP_REQ *req=NULL;
+  char *s=NULL;
+  TRP_RC rc=TRP_ERROR;
+
+  /* check message type and body type for agreement */
+  req=trp_req_new(tmp_ctx);
+  if (req==NULL) {
+    rc=TRP_NOMEM;
+    goto cleanup;
+  }
+
+  rc=tr_msg_get_json_string(jreq, "community", &s, tmp_ctx);
+  if (rc!=TRP_SUCCESS)
+    goto cleanup;
+  req->comm=tr_new_name(s);
+  talloc_free(s); s=NULL;
+
+  rc=tr_msg_get_json_string(jreq, "realm", &s, tmp_ctx);
+  if (rc!=TRP_SUCCESS)
+    goto cleanup;
+  req->realm=tr_new_name(s);
+  talloc_free(s); s=NULL;
+
+  rc=TRP_SUCCESS;
+  talloc_steal(mem_ctx, req);
+
+cleanup:
+  talloc_free(tmp_ctx);
+  if (rc!=TRP_SUCCESS)
+    return NULL;
+  return req;
 }
 
 char *tr_msg_encode(TR_MSG *msg) 
 {
   json_t *jmsg;
   json_t *jmsg_type;
+  char *encoded;
 
   /* TBD -- add error handling */
   jmsg = json_object();
@@ -273,56 +869,64 @@ char *tr_msg_encode(TR_MSG *msg)
   switch (msg->msg_type) 
     {
     case TID_REQUEST:
-      jmsg_type = json_string("TIDRequest");
+      jmsg_type = json_string("tid_request");
       json_object_set_new(jmsg, "msg_type", jmsg_type);
-      json_object_set_new(jmsg, "msg_body", tr_msg_encode_tidreq(msg->tid_req));
+      json_object_set_new(jmsg, "msg_body", tr_msg_encode_tidreq(tr_msg_get_req(msg)));
       break;
 
     case TID_RESPONSE:
-      jmsg_type = json_string("TIDResponse");
+      jmsg_type = json_string("tid_response");
       json_object_set_new(jmsg, "msg_type", jmsg_type);
-      json_object_set_new(jmsg, "msg_body", tr_msg_encode_tidresp(msg->tid_resp));
+      json_object_set_new(jmsg, "msg_body", tr_msg_encode_tidresp(tr_msg_get_resp(msg)));
       break;
 
-      /* TBD -- Add TR message types */
+    case TRP_UPDATE:
+      jmsg_type = json_string("trp_update");
+      json_object_set_new(jmsg, "msg_type", jmsg_type);
+      json_object_set_new(jmsg, "msg_body", tr_msg_encode_trp_upd(tr_msg_get_trp_upd(msg)));
+      break;
+
+    case TRP_REQUEST:
+      jmsg_type = json_string("trp_request");
+      json_object_set_new(jmsg, "msg_type", jmsg_type);
+      json_object_set_new(jmsg, "msg_body", tr_msg_encode_trp_req(tr_msg_get_trp_req(msg)));
+      break;
 
     default:
       json_decref(jmsg);
       return NULL;
     }
-  
-  return(json_dumps(jmsg, 0));
+
+  encoded=json_dumps(jmsg, 0);
+  json_decref(jmsg);
+  return encoded;
 }
 
 TR_MSG *tr_msg_decode(char *jbuf, size_t buflen)
 {
-  TR_MSG *msg;
+  TR_MSG *msg=NULL;
   json_t *jmsg = NULL;
   json_error_t rc;
-  size_t msize;
-  json_t *jtype;
-  json_t *jbody;
+  json_t *jtype=NULL;
+  json_t *jbody=NULL;
   const char *mtype = NULL;
 
-  if (NULL == (jmsg = json_loadb(jbuf, buflen, 0, &rc))) {
-    fprintf (stderr, "tr_msg_decode(): error loading object, rc = %d.\n", rc);
+  if (NULL == (jmsg = json_loadb(jbuf, buflen, JSON_DISABLE_EOF_CHECK, &rc))) {
+    tr_debug("tr_msg_decode(): error loading object");
     return NULL;
   }
 
   if (!(msg = malloc(sizeof(TR_MSG)))) {
-    fprintf (stderr, "tr_msg_decode(): Error allocating TR_MSG structure.\n");
+    tr_debug("tr_msg_decode(): Error allocating TR_MSG structure.");
     json_decref(jmsg);
     return NULL;
   }
  
   memset(msg, 0, sizeof(TR_MSG));
 
-  if ((2 != (msize = json_object_size(jmsg))) ||
-      (NULL == (jtype = json_object_get(jmsg, "msg_type"))) ||
-      (!json_is_string(jtype)) ||
-      (NULL == (jbody = json_object_get(jmsg, "msg_body"))) ||
-      (!json_is_object(jbody))) {
-    fprintf (stderr, "tr_msg_decode(): Error parsing message.\n");
+  if ((NULL == (jtype = json_object_get(jmsg, "msg_type"))) ||
+      (NULL == (jbody = json_object_get(jmsg, "msg_body")))) {
+    tr_debug("tr_msg_decode(): Error parsing message header.");
     json_decref(jmsg);
     tr_msg_free_decoded(msg);
     return NULL;
@@ -330,17 +934,25 @@ TR_MSG *tr_msg_decode(char *jbuf, size_t buflen)
 
   mtype = json_string_value(jtype);
 
-  if (0 == strcmp(mtype, "TIDRequest")) {
+  if (0 == strcmp(mtype, "tid_request")) {
     msg->msg_type = TID_REQUEST;
-    msg->tid_req = tr_msg_decode_tidreq(jbody);
+    tr_msg_set_req(msg, tr_msg_decode_tidreq(jbody));
   }
-  else if (0 == strcmp(mtype, "TIDResponse")) {
+  else if (0 == strcmp(mtype, "tid_response")) {
     msg->msg_type = TID_RESPONSE;
-    msg->tid_resp = tr_msg_decode_tidresp(jbody);
+    tr_msg_set_resp(msg, tr_msg_decode_tidresp(jbody));
+  }
+  else if (0 == strcmp(mtype, "trp_update")) {
+    msg->msg_type = TRP_UPDATE;
+    tr_msg_set_trp_upd(msg, tr_msg_decode_trp_upd(NULL, jbody)); /* null talloc context for now */
+  }
+  else if (0 == strcmp(mtype, "trp_request")) {
+    msg->msg_type = TRP_UPDATE;
+    tr_msg_set_trp_req(msg, tr_msg_decode_trp_req(NULL, jbody)); /* null talloc context for now */
   }
   else {
     msg->msg_type = TR_UNKNOWN;
-    msg->tid_req = NULL;
+    msg->msg_rep = NULL;
   }
   return msg;
 }
@@ -353,8 +965,24 @@ void tr_msg_free_encoded(char *jmsg)
 
 void tr_msg_free_decoded(TR_MSG *msg)
 {
-  if (msg)
+  if (msg) {
+    switch (msg->msg_type) {
+    case TID_REQUEST:
+      tid_req_free(tr_msg_get_req(msg));
+      break;
+    case TID_RESPONSE:
+      tid_resp_free(tr_msg_get_resp(msg));
+      break;
+    case TRP_UPDATE:
+      trp_upd_free(tr_msg_get_trp_upd(msg));
+      break;
+    case TRP_REQUEST:
+      trp_req_free(tr_msg_get_trp_req(msg));
+    default:
+      break;
+    }
     free (msg);
+  }
 }