farm server redirection segfault at session closing about freerdp

[celsius]

here is a patch that fixes the issue (loss of pointer "targetNetAddresses" because of redirection->targetNetAddresses++; ) :

diff --git a/libfreerdp-core/redirection.c b/libfreerdp-core/redirection.c
index 5e6c0e5..0f1e07d 100644
--- a/libfreerdp-core/redirection.c
+++ b/libfreerdp-core/redirection.c
@@ -138,12 +138,13 @@ boolean rdp_recv_server_redirection_pdu(rdpRdp* rdp, STREAM* s)

                redirection->targetNetAddresses = (rdpString*) xzalloc(count * sizeof(rdpString));

-               while (count > 0)
+
+               int i;
+
+               for (i=0; i < count; i++)
                {
-                       freerdp_string_read_length32(s, redirection->targetNetAddresses, rdp->settings->uniconv);
-                       DEBUG_REDIR("targetNetAddresses: %s", redirection->targetNetAddresses->ascii);
-                       redirection->targetNetAddresses++;
-                       count--;
+                       freerdp_string_read_length32(s, &redirection->targetNetAddresses[i], rdp->settings->uniconv);
+                       DEBUG_REDIR("targetNetAddresses: %s", (&redirection->targetNetAddresses[i])->ascii);
                }
        }

[celsius]

NB: the patch fixes one segfault but there are others happening after this one is corrected (notably in the freeing of the cache, rdp and xfi). These also happen only when logging out after a redirection, I'll try to look into it.