| Commit message (Collapse) | Author | Age |
|
|
|
| |
Based on a patch by Ray Link <rlink+git@cs.cmu.edu>
|
|
|
|
| |
zwrite: Manpage update for -S
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Eliminate compiler warnings due to various issues (listed below). This
allows Zephyr to build cleanly under GCC versions ranging from 4.1.0 to
4.7.2 with all of the options shown below:
-g -O2 -Wall -Werror
-Wno-deprecated-declarations
-Wmissing-declarations
-Wpointer-arith
-Wstrict-prototypes
-Wshadow
-Wextra
-Wno-missing-field-initializers
-Wno-unused-parameter
and, on recent versions, -Wunreachable-code
Test builds were done
- On Ubuntu 12.10 (Quantal Quetzal) using both MIT Kerberos 1.10.1 and
Heimdal 1.6, without krb4 and both with and without C-Ares and Hesiod
- On Fedora 14 using Heimdal 0.6, without C-Ares or Hesiod and both
with and without krb4 (KTH Kerberos 1.3rc2)
- On Fedora Core 3, Fedora Core 5, Fedora 7, and Fedora 10, using
Heimdal 0.6 and without C-Ares, Hesiod, or krb4
It also allows clean builds on Solaris 10 under the Sun Studio 12 (9/07)
C compiler with the following options:
-g -fd -v -errfmt -errhdr=%user -errtags=yes -errwarn=%all
-erroff=E_OLD_STYLE_FUNC_DECL,E_ENUM_TYPE_MISMATCH_ARG,E_ARG_INCOMPATIBLE_WITH_ARG
... and under Solaris 9 with the Sun Forte 7 (3/02) C compiler with the above
options and -erroff=E_FUNC_HAS_NO_RETURN_STMT. Solaris builds were done
with Heimdal 0.6 and without C-Ares, Hesiod, or krb4.
The following types of issues are addressed in this change:
- Parameters and local variables with the same names as library functions
- Parameters and local variables with the same names as globals
- Declarations for exported global variables missing from headers
- Prototypes for exported functions missing from headers
- Missing 'static' on functions that shouldn't be exported
- Old-style function declarations
- Duplicate declarations
- Type mismatches
- Unused variables and functions
- Uninitialized variables
- Forward references to enums
- Necessary header files not included
- Violations of the aliasing rules, where GCC was able to detect them
- Missing braces on if blocks that might be empty
- Attempts to do pointer arithmetic on pointers of type void *, which
is not permitted in standard C.
- An attempt to pass a function pointer via a void * parameter, which is
not permitted in standard C. Instead, we now pass a pointer to a
structure, which then contains the required function pointer.
- Unnecessary inclusion of <krb5_err.h>, which is already included by
<krb5.h> when the former exists, and might not be protected against
double inclusion, depending on which com_err was used.
- Missing include of <com_err.h>, which was masked by the fact that it is
included by headers generated by e2fsprogs compile_et
- Use of com_err() with a non-constant value in place of the format string,
which in every case was a fixed-size buffer in which a message was built
using sprintf(!). Both the calls to sprintf and the fixed-size buffers
have been removed, in favor of just letting com_err() do the formatting.
- Various cases where X library functions expecting a parameter of type
wchar_t * were instead passed a parameter of type XChar2b *. The two
types look similar, but are not the same and are _not_ interchangeable.
- An overly-simplistic configure test which failed to detect existence of
<term.h> on Solaris, due to not including <curses.h>.
- Using the wrong type for the flags output of krb5_auth_con_getflags()
when building against Heimdal. A configure test is added to detect
the correct type.
|
|
|
|
|
|
| |
We need the bytes, no modern client uses it, and it's inherently a
security vulnerability. For those clients that do use it, provide a link
to a page on the zephyr wiki that explains the issue.
|
|
|
|
|
| |
i.e. don't keep generated or foreign stuff in our source tree.
As a side effect, this lets us use a libtool, etc. from this century
|
|
|
|
| |
Fixes #72
|
|
|
|
| |
wthrowe@mit.edu
|
|
|
|
|
| |
Don't free the incoming notice if we don't know for sure there's one.
Drop the message on read error in the tty case.
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Rearrange what libraries get pulled in where, so dpkg-shlibdeps
doesn't whine (see a theme here?) (Also so that, say, znol, doesn't
have to link with Everything.)
This will break platforms that don't do shared-library dependencies.
|
| |
|
| |
|
|
|
|
| |
getsid problem
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
targets, even if the loop body does a "continue".
|
| |
|
|
|
|
|
| |
know if ^C'ing might not work. (Based on a patch by Greg.) Also, put more
useful information about class/instance in the message.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Rip out support for in-tree com_err and ss.
|
|
|
|
| |
detailed change information.
|
| |
|
|
|
|
| |
Maybe more.
|
| |
|
|
|
|
| |
and class/instance zephyrgrams hang.
|
| |
|
| |
|
|
|
|
|
| |
Changed bcopy to memcpy, index to strchr [ANSI]
Set "auth" variable to ZAUTH for -a (in case -d was also specified)
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
from a terminal.
|
| |
|
|
|
|
|
| |
Also made character arrays in main static (and therefore automatically
cleared).
|
| |
|
| |
|
|
|
|
| |
a reasonable set of defaults are agreed upon.
|
| |
|
| |
|
| |
|
|
|
|
| |
add -s to usage message
|