Alan Coopersmith
50b1097643
Constify the reason string throughout the authorization check framework
...
Almost all of the places the string is assigned point to a literal
string constant, so use const char * for those, and const char **
for function calls that return it via an argument. Fortunately
the top level function, ClientAuthorized, which returns the string
as its return value is called from only one place, ProcEstablishConnection.
ProcEstablishConnection stores either that return value or a string literal
in char *reason. It only uses reason as an argument to SendConnSetup.
SendConnSetup passes the reason argument to strlen & WriteToClient,
both of which already have const qualifiers on their args.
Thus added const to the reason variable in ProcEstablishConnection
and the reason argument to SendConnSetup.
Fixes gcc warnings:
dispatch.c: In function 'ProcEstablishConnection':
dispatch.c:3711:9: warning: assignment discards qualifiers from pointer target type
auth.c: In function 'CheckAuthorization':
auth.c:218:14: warning: assignment discards qualifiers from pointer target type
auth.c:220:20: warning: assignment discards qualifiers from pointer target type
connection.c: In function 'ClientAuthorized':
connection.c:683:3: warning: return discards qualifiers from pointer target type
mitauth.c: In function 'MitCheckCookie':
mitauth.c:88:13: warning: assignment discards qualifiers from pointer target type
xdmauth.c:259:14: warning: assignment discards qualifiers from pointer target type
xdmauth.c:270:14: warning: assignment discards qualifiers from pointer target type
xdmauth.c:277:11: warning: assignment discards qualifiers from pointer target type
xdmauth.c:293:15: warning: assignment discards qualifiers from pointer target type
xdmauth.c:313:14: warning: assignment discards qualifiers from pointer target type
xdmauth.c:322:11: warning: assignment discards qualifiers from pointer target type
rpcauth.c: In function 'SecureRPCCheck':
rpcauth.c:136:10: warning: assignment discards qualifiers from pointer target type
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Jamey Sharp <jamey@minilop.net>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
2011-12-12 17:03:10 -08:00
..
.gitignore
.gitignore: use common defaults with custom section #24239
2009-11-11 21:40:20 -08:00
callback.h
dix: optimize CallCallbacks
2010-10-13 09:24:06 -07:00
client.h
os: Add facilities for client ID tracking.
2010-12-23 13:53:18 +02:00
closestr.h
dixfonts: Deobfuscate GC ops calls.
2010-09-13 15:55:17 -07:00
closure.h
colormap.h
Quit using clientErrorValue in dix/colormap.c.
2010-05-13 17:14:07 -07:00
colormapst.h
Revert "Bug #6924 : Restore the ABI for DrawableRec and ColormapRec to the state"
2011-09-19 22:31:21 -07:00
cursor.h
Eliminate MAXSCREENS-sized CursorScreenKey array.
2011-09-19 22:53:05 -07:00
cursorstr.h
Change the devPrivates API to require dixRegisterPrivateKey
2010-06-05 19:23:03 -07:00
dix-config-apple-verbatim.h
Move the apple fat binary hacks back to a header file, and make it apple-only.
2009-01-30 16:36:45 -08:00
dix-config.h.in
Move to autoconf standard function name checks & defines
2011-12-05 14:32:45 -08:00
dix.h
Merge remote-tracking branch 'whot/for-keith'
2011-12-08 20:57:26 -08:00
dixaccess.h
dix: add a new DixAccess bit, "DixPostAccess".
2009-06-26 17:05:52 -04:00
dixevents.h
Rework symbol visibility for easier maintenance
2008-12-03 05:43:34 -02:00
dixfont.h
dix: use one single function to register fpe fonts
2010-06-24 13:55:28 -07:00
dixfontstr.h
Remove #define NEED_EVENTS and NEED_REPLIES
2008-12-12 11:43:32 +10:00
dixgrabs.h
dix: add CopyGrab() function
2011-12-09 14:56:23 +10:00
dixstruct.h
Save major/minor opcodes in ClientRec for RecordAReply
2011-12-01 15:11:20 +00:00
eventconvert.h
Input: Allow EventToCore to return multiple events
2011-02-17 09:40:59 +10:00
events.h
XQuartz: Transition from xEvent based mieq to InternalEvent
2009-09-27 23:31:07 -07:00
eventstr.h
dix: add ScrollInfo to DeviceChangedEvents
2011-10-25 14:06:41 +10:00
exevents.h
Make XIGetKnownProperty take a const char * argument
2011-11-23 12:15:06 -08:00
extension.h
Save major/minor opcodes in ClientRec for RecordAReply
2011-12-01 15:11:20 +00:00
extinit.h
Fix gcc -Wwrite-strings warnings in various extensions
2011-11-23 12:15:07 -08:00
extnsionst.h
Fix gcc -Wwrite-strings warnings in various extensions
2011-11-23 12:15:07 -08:00
gc.h
Un-export CreateScratchGC now that nothing external uses it.
2010-09-13 15:58:08 -07:00
gcstruct.h
Delete unused lastWinOrg field from GCs.
2010-09-13 15:55:17 -07:00
globals.h
config: add udev/systemd multi-seat support
2011-08-22 15:56:49 +10:00
hotplug.h
Rework symbol visibility for easier maintenance
2008-12-03 05:43:34 -02:00
input.h
Add a new XI2Mask struct and a few helper functions.
2011-12-09 14:56:23 +10:00
inputstr.h
input: swap the server over to use the XI2mask struct
2011-12-09 14:56:23 +10:00
inpututils.h
Add a new XI2Mask struct and a few helper functions.
2011-12-09 14:56:23 +10:00
kdrive-config.h.in
kdrive: Add option to compile out input drivers
2010-01-06 19:47:53 +06:00
list.h
include: rename "foos" to "list_of_foos" in the struct list documentation
2011-12-06 18:15:15 +10:00
Makefile.am
Add 'optionstr.h' to include/Makefile.am
2011-11-20 14:50:57 -08:00
misc.h
include: add BUG_WARN_MSG for custom error message on bug condition
2011-12-09 14:56:24 +10:00
miscstruct.h
opaque.h
dix: adds support for none root window background
2010-11-11 23:24:43 +02:00
optionstr.h
input: switch InputOption to use XF86OptionRec storage.
2011-10-25 14:06:39 +10:00
os.h
Constify the reason string throughout the authorization check framework
2011-12-12 17:03:10 -08:00
pixmap.h
dix: Remove the memory of the multibuffer extension
2010-10-13 09:18:41 -07:00
pixmapstr.h
Revert "dix: Remove usage_hint from pixmaps, store it in ->drawable.class"
2011-03-17 23:48:52 -07:00
privates.h
"privates.h", line 198: warning: void function cannot return value
2011-05-27 19:52:10 -07:00
property.h
Rework symbol visibility for easier maintenance
2008-12-03 05:43:34 -02:00
propertyst.h
dix: Shrink PropertyRec on LP64
2011-03-14 13:44:46 -04:00
protocol-versions.h
Support (and require) XI 2.1
2011-09-29 14:09:41 +10:00
ptrveloc.h
Input: Widen pointer acceleration types to double
2011-09-29 12:24:33 +10:00
region.h
regionstr.h
Add RegionInitBoxes(), and fix some buggy callers of RegionInit().
2011-04-22 21:39:31 -04:00
registry.h
constify strings in resource name registry
2011-12-12 17:03:08 -08:00
resource.h
constify strings in resource name registry
2011-12-12 17:03:08 -08:00
rgb.h
screenint.h
Rework symbol visibility for easier maintenance
2008-12-03 05:43:34 -02:00
scrnintstr.h
Store desktop dimensions in screenInfo.
2011-10-13 11:26:09 +10:00
selection.h
Rework symbol visibility for easier maintenance
2008-12-03 05:43:34 -02:00
servermd.h
xfree86: nds32: add nds32 related definitions into include headers.
2010-09-23 15:35:48 -07:00
site.h
dix: delete logo hack screen saver
2010-11-11 23:20:35 +02:00
swaprep.h
Rework symbol visibility for easier maintenance
2008-12-03 05:43:34 -02:00
swapreq.h
Rework symbol visibility for easier maintenance
2008-12-03 05:43:34 -02:00
validate.h
version-config.h.in
Move VENDOR_* defines from AC_SUBST to a header to avoid angering shave.
2009-04-14 10:35:44 -04:00
window.h
DIX: Make PrintWindowTree actually useful
2011-07-01 08:46:15 +10:00
windowstr.h
composite: Inhibit window background paint with manual subwindow redirection
2011-08-03 20:46:36 -07:00
XIstubs.h
input: Purge AddOtherInputDevices DDX hook.
2010-09-01 15:26:52 +10:00
xkb-config.h.in
Move to autoconf standard function name checks & defines
2011-12-05 14:32:45 -08:00
xkbfile.h
Mark XKB char * as const to clean up gcc -Wwrite-strings warnings
2011-11-23 12:15:06 -08:00
xkbrules.h
XKB: Remove descriptions from maprules
2009-01-22 15:08:56 +11:00
xkbsrv.h
Mark XKB char * as const to clean up gcc -Wwrite-strings warnings
2011-11-23 12:15:06 -08:00
xkbstr.h
Revert "XKB: Sanitise * actions" commits ( #19602 )
2009-07-10 11:23:52 +10:00
xorg-config.h.in
configure: wrap PCI code with macro and set it at build time
2011-10-15 21:18:47 -07:00
xorg-server.h.in
Fix gcc warnings about redundant declarations of fallback functions
2011-12-05 14:32:59 -08:00
Xprintf.h
Add asprintf() implementation for platforms without it
2010-12-07 11:10:35 -08:00
xserver-properties.h
Input: Add vertical and horizontal scroll axes
2011-09-29 12:26:43 +10:00
xsha1.h
Move SHA1 computation from render/glyph.c to os/
2009-10-15 01:17:44 +02:00
xwin-config.h.in
Remove unused function checks from configure.ac & include/*.h.in
2011-12-03 10:19:12 -08:00