Make a channel getter method to retrieve transport names.

1 files changed, 7 insertions, 2 deletions

diff --git a/src/or/channel.c b/src/or/channel.c
index 4b6c7e107d..ea5f9610a0 100644
--- a/src/or/channel.c
+++ b/src/or/channel.c
@@ -2379,9 +2379,14 @@ channel_do_open_actions(channel_t *chan)
     /* only report it to the geoip module if it's not a known router */
     if (!router_get_by_id_digest(chan->identity_digest)) {
       if (channel_get_addr_if_possible(chan, &remote_addr)) {
-        /* XXXX 5040/4773 : Is this 'NULL' right? */
-        geoip_note_client_seen(GEOIP_CLIENT_CONNECT, &remote_addr, NULL,
+        char *transport_name = NULL;
+        if (chan->get_transport_name(chan, &transport_name) < 0)
+          transport_name = NULL;
+
+        geoip_note_client_seen(GEOIP_CLIENT_CONNECT,
+                               &remote_addr, transport_name,
                                now);
+        tor_free(transport_name);
       }
       /* Otherwise the underlying transport can't tell us this, so skip it */
     }