Rewrite code to avoid triggering warning about casting param to free()

Since free() takes a void* parameters do not need to be cast.

The existing code here is actally fine, but it trips up the
syntax-check rule, so tweak it to an equivalent construct
which passes the syntax check

1 files changed, 2 insertions, 1 deletions

diff --git a/server/smartcard.c b/server/smartcard.c
index 43ca1ac4..facc4b37 100644
--- a/server/smartcard.c
+++ b/server/smartcard.c
@@ -335,7 +335,8 @@ static void smartcard_channel_release_pipe_item(RedChannelClient *rcc,
                                       PipeItem *item, int item_pushed)
 {
     if (item->type == PIPE_ITEM_TYPE_MSG) {
-        free(((MsgItem*)item)->vheader);
+        MsgItem *mi = (MsgItem *)item;
+        free(mi->vheader);
     }
     free(item);
 }