Include com_err.h in dumpstuff.c

src/volser/dumpstuff.c uses afs_error_message in its error reporting,
but it does not include com_err.h, so it does not get the prototype
for afs_error_message. The default return type for unprototyped
functions is 'int', so the return value of afs_error_message can get
truncated on platforms where sizeof(char*) > sizeof(int), causing a
segfault whenever the result of afs_error_message is logged.

This is a 1.4-only change. This was fixed in the 1.5 branch in the
midst of the larger warning cleanup effort.

Change-Id: Id3bcde20609130d8e4e393c6c8aae75682bcd966
Reviewed-on: http://gerrit.openafs.org/2526
Tested-by: Andrew Deason <adeason@sinenomine.net>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
(cherry picked from commit ebb6accf18cf3eeb6a99025ccf5228f979463d0e)

diff --git a/src/volser/dumpstuff.c b/src/volser/dumpstuff.c
index f8ff68d2c7015496de2f2635015b6623f12d5d59..69892548c18a01eb1462f98b4c3909b6613e73ff 100644 (file)
--- a/src/volser/dumpstuff.c
+++ b/src/volser/dumpstuff.c
@@ -45,6 +45,7 @@
 #include "dump.h"
 #include <afs/fssync.h>
 #include <afs/acl.h>
+#include <afs/com_err.h>
 #include "volser.h"
 #include "volint.h"