[PATCH:xdm 3/5] Stop casting free() arguments to (char *)

Mark Kettenis mark.kettenis at xs4all.nl
Sat May 14 12:12:04 PDT 2011


> From: Alan Coopersmith <alan.coopersmith at oracle.com>
> Date: Sat, 14 May 2011 11:01:31 -0700
> 
> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

Heh, noticed the stupid casts in one of the other diffs in this
series, and asked myself why you didn't remove them.  So obviously I
agree with this ;).

Reviewed-by: Mark Kettenis <kettenis at openbsd.org>

>  chooser/chooser.c |   12 ++++++------
>  xdm/access.c      |   18 +++++++++---------
>  xdm/auth.c        |   14 +++++++-------
>  xdm/choose.c      |   12 ++++++------
>  xdm/dpylist.c     |    8 ++++----
>  xdm/file.c        |    4 ++--
>  xdm/krb5auth.c    |    6 +++---
>  xdm/mitauth.c     |   12 ++++++------
>  xdm/protodpy.c    |   10 +++++-----
>  xdm/rpcauth.c     |    6 +++---
>  xdm/server.c      |    2 +-
>  xdm/util.c        |    4 ++--
>  xdm/xdmauth.c     |   18 +++++++++---------
>  xdm/xdmcp.c       |   12 ++++++------
>  14 files changed, 69 insertions(+), 69 deletions(-)


More information about the xorg-devel mailing list