diff options
author | Keith Packard <keithp@keithp.com> | 2012-03-21 12:55:09 -0700 |
---|---|---|
committer | Keith Packard <keithp@keithp.com> | 2012-03-21 13:54:42 -0700 |
commit | 9838b7032ea9792bec21af424c53c07078636d21 (patch) | |
tree | b72d0827dac50f0f3b8eab29b3b7639546d735d7 /os/mitauth.c | |
parent | 75199129c603fc8567185ac31866c9518193cb78 (diff) |
Introduce a consistent coding style
This is strictly the application of the script 'x-indent-all.sh'
from util/modular. Compared to the patch that Daniel posted in
January, I've added a few indent flags:
-bap
-psl
-T PrivatePtr
-T pmWait
-T _XFUNCPROTOBEGIN
-T _XFUNCPROTOEND
-T _X_EXPORT
The typedefs were needed to make the output of sdksyms.sh match the
previous output, otherwise, the code is formatted badly enough that
sdksyms.sh generates incorrect output.
The generated code was compared with the previous version and found to
be essentially identical -- "assert" line numbers and BUILD_TIME were
the only differences found.
The comparison was done with this script:
dir1=$1
dir2=$2
for dir in $dir1 $dir2; do
(cd $dir && find . -name '*.o' | while read file; do
dir=`dirname $file`
base=`basename $file .o`
dump=$dir/$base.dump
objdump -d $file > $dump
done)
done
find $dir1 -name '*.dump' | while read dump; do
otherdump=`echo $dump | sed "s;$dir1;$dir2;"`
diff -u $dump $otherdump
done
Signed-off-by: Keith Packard <keithp@keithp.com>
Acked-by: Daniel Stone <daniel@fooishbar.org>
Acked-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Diffstat (limited to 'os/mitauth.c')
-rw-r--r-- | os/mitauth.c | 151 |
1 files changed, 66 insertions, 85 deletions
diff --git a/os/mitauth.c b/os/mitauth.c index 477736ee8..768a52a22 100644 --- a/os/mitauth.c +++ b/os/mitauth.c @@ -41,27 +41,24 @@ from The Open Group. #include "dixstruct.h" static struct auth { - struct auth *next; - unsigned short len; - char *data; - XID id; + struct auth *next; + unsigned short len; + char *data; + XID id; } *mit_auth; int -MitAddCookie ( - unsigned short data_length, - const char *data, - XID id) +MitAddCookie(unsigned short data_length, const char *data, XID id) { - struct auth *new; + struct auth *new; - new = malloc(sizeof (struct auth)); + new = malloc(sizeof(struct auth)); if (!new) - return 0; + return 0; new->data = malloc((unsigned) data_length); if (!new->data) { - free(new); - return 0; + free(new); + return 0; } new->next = mit_auth; mit_auth = new; @@ -72,126 +69,110 @@ MitAddCookie ( } XID -MitCheckCookie ( - unsigned short data_length, - const char *data, - ClientPtr client, - const char **reason) +MitCheckCookie(unsigned short data_length, + const char *data, ClientPtr client, const char **reason) { - struct auth *auth; + struct auth *auth; - for (auth = mit_auth; auth; auth=auth->next) { + for (auth = mit_auth; auth; auth = auth->next) { if (data_length == auth->len && - memcmp (data, auth->data, (int) data_length) == 0) - return auth->id; + memcmp(data, auth->data, (int) data_length) == 0) + return auth->id; } *reason = "Invalid MIT-MAGIC-COOKIE-1 key"; return (XID) -1; } int -MitResetCookie (void) +MitResetCookie(void) { - struct auth *auth, *next; + struct auth *auth, *next; - for (auth = mit_auth; auth; auth=next) { - next = auth->next; - free(auth->data); - free(auth); + for (auth = mit_auth; auth; auth = next) { + next = auth->next; + free(auth->data); + free(auth); } mit_auth = 0; return 0; } XID -MitToID ( - unsigned short data_length, - char *data) +MitToID(unsigned short data_length, char *data) { - struct auth *auth; + struct auth *auth; - for (auth = mit_auth; auth; auth=auth->next) { - if (data_length == auth->len && - memcmp (data, auth->data, data_length) == 0) - return auth->id; + for (auth = mit_auth; auth; auth = auth->next) { + if (data_length == auth->len && + memcmp(data, auth->data, data_length) == 0) + return auth->id; } return (XID) -1; } int -MitFromID ( - XID id, - unsigned short *data_lenp, - char **datap) +MitFromID(XID id, unsigned short *data_lenp, char **datap) { - struct auth *auth; - - for (auth = mit_auth; auth; auth=auth->next) { - if (id == auth->id) { - *data_lenp = auth->len; - *datap = auth->data; - return 1; - } + struct auth *auth; + + for (auth = mit_auth; auth; auth = auth->next) { + if (id == auth->id) { + *data_lenp = auth->len; + *datap = auth->data; + return 1; + } } return 0; } int -MitRemoveCookie ( - unsigned short data_length, - const char *data) +MitRemoveCookie(unsigned short data_length, const char *data) { - struct auth *auth, *prev; + struct auth *auth, *prev; prev = 0; - for (auth = mit_auth; auth; prev = auth, auth=auth->next) { - if (data_length == auth->len && - memcmp (data, auth->data, data_length) == 0) - { - if (prev) - prev->next = auth->next; - else - mit_auth = auth->next; - free(auth->data); - free(auth); - return 1; - } + for (auth = mit_auth; auth; prev = auth, auth = auth->next) { + if (data_length == auth->len && + memcmp(data, auth->data, data_length) == 0) { + if (prev) + prev->next = auth->next; + else + mit_auth = auth->next; + free(auth->data); + free(auth); + return 1; + } } return 0; } #ifdef XCSECURITY -static char cookie[16]; /* 128 bits */ +static char cookie[16]; /* 128 bits */ XID -MitGenerateCookie ( - unsigned data_length, - const char *data, - XID id, - unsigned *data_length_return, - char **data_return) +MitGenerateCookie(unsigned data_length, + const char *data, + XID id, unsigned *data_length_return, char **data_return) { int i = 0; int status; - while (data_length--) - { - cookie[i++] += *data++; - if (i >= sizeof (cookie)) i = 0; + while (data_length--) { + cookie[i++] += *data++; + if (i >= sizeof(cookie)) + i = 0; } - GenerateRandomData(sizeof (cookie), cookie); - status = MitAddCookie(sizeof (cookie), cookie, id); - if (!status) - { - id = -1; + GenerateRandomData(sizeof(cookie), cookie); + status = MitAddCookie(sizeof(cookie), cookie, id); + if (!status) { + id = -1; } - else - { - *data_return = cookie; - *data_length_return = sizeof (cookie); + else { + *data_return = cookie; + *data_length_return = sizeof(cookie); } return id; } -#endif /* XCSECURITY */ +#endif /* XCSECURITY */ |