rxrpc: Provide a way for AFS to ask for the peer address of a call
authorDavid Howells <dhowells@redhat.com>
Tue, 30 Aug 2016 08:49:29 +0000 (09:49 +0100)
committerDavid Howells <dhowells@redhat.com>
Tue, 30 Aug 2016 15:07:53 +0000 (16:07 +0100)
Provide a function so that kernel users, such as AFS, can ask for the peer
address of a call:

   void rxrpc_kernel_get_peer(struct rxrpc_call *call,
      struct sockaddr_rxrpc *_srx);

In the future the kernel service won't get sk_buffs to look inside.
Further, this allows us to hide any canonicalisation inside AF_RXRPC for
when IPv6 support is added.

Also propagate this through to afs_find_server() and issue a warning if we
can't handle the address family yet.

Signed-off-by: David Howells <dhowells@redhat.com>
Documentation/networking/rxrpc.txt
fs/afs/cmservice.c
fs/afs/internal.h
fs/afs/rxrpc.c
fs/afs/server.c
include/net/af_rxrpc.h
net/rxrpc/peer_object.c

index 70c926ae212d397c8543cbc9da56ad815fe62da5..dfe0b008df743bccadad1a53c1f0a561b94c1863 100644 (file)
@@ -868,6 +868,13 @@ The kernel interface functions are as follows:
      This is used to allocate a null RxRPC key that can be used to indicate
      anonymous security for a particular domain.
 
+ (*) Get the peer address of a call.
+
+       void rxrpc_kernel_get_peer(struct socket *sock, struct rxrpc_call *call,
+                                  struct sockaddr_rxrpc *_srx);
+
+     This is used to find the remote peer address of a call.
+
 
 =======================
 CONFIGURABLE PARAMETERS
index ca32d891bbc3872b6a82fce9a206bc86d06c27b0..77ee481059ac4cef73c6209a86ce0724c7f14ab5 100644 (file)
@@ -167,9 +167,9 @@ static void SRXAFSCB_CallBack(struct work_struct *work)
 static int afs_deliver_cb_callback(struct afs_call *call, struct sk_buff *skb,
                                   bool last)
 {
+       struct sockaddr_rxrpc srx;
        struct afs_callback *cb;
        struct afs_server *server;
-       struct in_addr addr;
        __be32 *bp;
        u32 tmp;
        int ret, loop;
@@ -178,6 +178,7 @@ static int afs_deliver_cb_callback(struct afs_call *call, struct sk_buff *skb,
 
        switch (call->unmarshall) {
        case 0:
+               rxrpc_kernel_get_peer(afs_socket, call->rxcall, &srx);
                call->offset = 0;
                call->unmarshall++;
 
@@ -282,8 +283,7 @@ static int afs_deliver_cb_callback(struct afs_call *call, struct sk_buff *skb,
 
        /* we'll need the file server record as that tells us which set of
         * vnodes to operate upon */
-       memcpy(&addr, &ip_hdr(skb)->saddr, 4);
-       server = afs_find_server(&addr);
+       server = afs_find_server(&srx);
        if (!server)
                return -ENOTCONN;
        call->server = server;
@@ -314,12 +314,14 @@ static int afs_deliver_cb_init_call_back_state(struct afs_call *call,
                                               struct sk_buff *skb,
                                               bool last)
 {
+       struct sockaddr_rxrpc srx;
        struct afs_server *server;
-       struct in_addr addr;
        int ret;
 
        _enter(",{%u},%d", skb->len, last);
 
+       rxrpc_kernel_get_peer(afs_socket, call->rxcall, &srx);
+
        ret = afs_data_complete(call, skb, last);
        if (ret < 0)
                return ret;
@@ -329,8 +331,7 @@ static int afs_deliver_cb_init_call_back_state(struct afs_call *call,
 
        /* we'll need the file server record as that tells us which set of
         * vnodes to operate upon */
-       memcpy(&addr, &ip_hdr(skb)->saddr, 4);
-       server = afs_find_server(&addr);
+       server = afs_find_server(&srx);
        if (!server)
                return -ENOTCONN;
        call->server = server;
@@ -347,11 +348,13 @@ static int afs_deliver_cb_init_call_back_state3(struct afs_call *call,
                                                struct sk_buff *skb,
                                                bool last)
 {
+       struct sockaddr_rxrpc srx;
        struct afs_server *server;
-       struct in_addr addr;
 
        _enter(",{%u},%d", skb->len, last);
 
+       rxrpc_kernel_get_peer(afs_socket, call->rxcall, &srx);
+
        /* There are some arguments that we ignore */
        afs_data_consumed(call, skb);
        if (!last)
@@ -362,8 +365,7 @@ static int afs_deliver_cb_init_call_back_state3(struct afs_call *call,
 
        /* we'll need the file server record as that tells us which set of
         * vnodes to operate upon */
-       memcpy(&addr, &ip_hdr(skb)->saddr, 4);
-       server = afs_find_server(&addr);
+       server = afs_find_server(&srx);
        if (!server)
                return -ENOTCONN;
        call->server = server;
index df976b2a7f40fbccf708e572372e41098b31f43f..d97552de9c59cf028dcfd5b0a9120c2d3be05fcf 100644 (file)
@@ -20,6 +20,7 @@
 #include <linux/sched.h>
 #include <linux/fscache.h>
 #include <linux/backing-dev.h>
+#include <net/af_rxrpc.h>
 
 #include "afs.h"
 #include "afs_vl.h"
@@ -607,6 +608,8 @@ extern void afs_proc_cell_remove(struct afs_cell *);
 /*
  * rxrpc.c
  */
+extern struct socket *afs_socket;
+
 extern int afs_open_socket(void);
 extern void afs_close_socket(void);
 extern void afs_data_consumed(struct afs_call *, struct sk_buff *);
@@ -654,7 +657,7 @@ do {                                                                \
 
 extern struct afs_server *afs_lookup_server(struct afs_cell *,
                                            const struct in_addr *);
-extern struct afs_server *afs_find_server(const struct in_addr *);
+extern struct afs_server *afs_find_server(const struct sockaddr_rxrpc *);
 extern void afs_put_server(struct afs_server *);
 extern void __exit afs_purge_servers(void);
 
index 14d04c848465a68f2b107c87b22b90f69062e9c5..a1916750e2f9a529bdaeb8fd51b0da514b48ad00 100644 (file)
@@ -16,7 +16,7 @@
 #include "internal.h"
 #include "afs_cm.h"
 
-static struct socket *afs_socket; /* my RxRPC socket */
+struct socket *afs_socket; /* my RxRPC socket */
 static struct workqueue_struct *afs_async_calls;
 static atomic_t afs_outstanding_calls;
 static atomic_t afs_outstanding_skbs;
index f342acf3547d0d06148335df01ab922939876b02..d4066ab7dd5505b364a6506a1a2d932274bb5d9d 100644 (file)
@@ -178,13 +178,18 @@ server_in_two_cells:
 /*
  * look up a server by its IP address
  */
-struct afs_server *afs_find_server(const struct in_addr *_addr)
+struct afs_server *afs_find_server(const struct sockaddr_rxrpc *srx)
 {
        struct afs_server *server = NULL;
        struct rb_node *p;
-       struct in_addr addr = *_addr;
+       struct in_addr addr = srx->transport.sin.sin_addr;
 
-       _enter("%pI4", &addr.s_addr);
+       _enter("{%d,%pI4}", srx->transport.family, &addr.s_addr);
+
+       if (srx->transport.family != AF_INET) {
+               WARN(true, "AFS does not yes support non-IPv4 addresses\n");
+               return NULL;
+       }
 
        read_lock(&afs_servers_lock);
 
index 7b0f88699b25eafceeca6a52da908ee03aa70d95..f9224e835d438cecfae8824d23a14eb0e69d17a8 100644 (file)
@@ -49,5 +49,7 @@ int rxrpc_kernel_get_error_number(struct sk_buff *);
 void rxrpc_kernel_free_skb(struct sk_buff *);
 struct rxrpc_call *rxrpc_kernel_accept_call(struct socket *, unsigned long);
 int rxrpc_kernel_reject_call(struct socket *);
+void rxrpc_kernel_get_peer(struct socket *, struct rxrpc_call *,
+                          struct sockaddr_rxrpc *);
 
 #endif /* _NET_RXRPC_H */
index 538e9831c69907b8d5a1756a77dc83ebfa2eff18..aebc73ac16dcfbd08321c18d46f9aa57e8cfcd05 100644 (file)
@@ -313,3 +313,18 @@ void __rxrpc_put_peer(struct rxrpc_peer *peer)
 
        kfree_rcu(peer, rcu);
 }
+
+/**
+ * rxrpc_kernel_get_peer - Get the peer address of a call
+ * @sock: The socket on which the call is in progress.
+ * @call: The call to query
+ * @_srx: Where to place the result
+ *
+ * Get the address of the remote peer in a call.
+ */
+void rxrpc_kernel_get_peer(struct socket *sock, struct rxrpc_call *call,
+                          struct sockaddr_rxrpc *_srx)
+{
+       *_srx = call->peer->srx;
+}
+EXPORT_SYMBOL(rxrpc_kernel_get_peer);