SUNRPC: Clean up functions that free address_strings array

Clean up: document the rule (kfree) and the exceptions
(RPC_DISPLAY_PROTO and RPC_DISPLAY_NETID) when freeing the objects in
a transport's address_strings array.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>

1 files changed, 10 insertions, 6 deletions

diff --git a/net/sunrpc/xprtrdma/transport.c b/net/sunrpc/xprtrdma/transport.c
index d1389afc8342..02c522c17de5 100644
--- a/net/sunrpc/xprtrdma/transport.c
+++ b/net/sunrpc/xprtrdma/transport.c
@@ -212,12 +212,16 @@ xprt_rdma_format_addresses(struct rpc_xprt *xprt)
 static void
 xprt_rdma_free_addresses(struct rpc_xprt *xprt)
 {
-        kfree(xprt->address_strings[RPC_DISPLAY_ADDR]);
-        kfree(xprt->address_strings[RPC_DISPLAY_PORT]);
-        kfree(xprt->address_strings[RPC_DISPLAY_ALL]);
-        kfree(xprt->address_strings[RPC_DISPLAY_HEX_ADDR]);
-        kfree(xprt->address_strings[RPC_DISPLAY_HEX_PORT]);
-        kfree(xprt->address_strings[RPC_DISPLAY_UNIVERSAL_ADDR]);
+        unsigned int i;
+
+        for (i = 0; i < RPC_DISPLAY_MAX; i++)
+                switch (i) {
+                case RPC_DISPLAY_PROTO:
+                case RPC_DISPLAY_NETID:
+                        continue;
+                default:
+                        kfree(xprt->address_strings[i]);
+                }
 }
 
 static void