| Commit message (Collapse) | Author | Age |
| |
|
| |
|
| |
|
|
|
|
| |
Looks like it's the same as free right now, but may as well call the right one.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
| |
Fix a leak in which we fail to free a Kerberos authentication context
in ZCheckSrvAuthentication if getting or setting the context flags fails.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
turns out that derived-key stuff actually worked if you were using heimdal.
|
| |
|
|
|
|
| |
(also fiddle around with what krb4 checksums are available in krb5-only land)
|
|
|
|
|
|
| |
keyusage stuff
such that it actually works.
|
|
|
|
|
|
|
|
|
| |
Turn off the replay cache (at least with MIT krb5 1.6)
All three authentication checks:
Initialize cksum1_buf so gcc will shut up.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
be NULL, something Wrong^TM, or off the end of z_other_fields.
bad all around.
|
|
|
|
|
| |
shouldn't be able to deal with incoming old-style packets
|
| |
|
|
|
|
|
|
|
| |
MAX_PRINCIPAL_SIZE
des hack
|
| |
|
| |
|
| |
|
|
|
|
| |
getsid problem
|
| |
|
|
|
|
| |
and have a command line switch to choose between the sort you use when getting them
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
all operations yet.
Also revert rev 1.23, because I got confused. We'll reintroduce it
shortly.
|
|
|
|
|
| |
* Better interrealm support: be prepared to try old and new style
checksumming, and make NOENCRYPTION case work.
|
|
|
|
|
| |
checksums on messages. With this change, you can't fiddle with subs if
you don't have a valid checksum.
|
| |
|
| |
|
|
|
|
| |
detailed change information.
|
|
|
|
| |
was expected by krb_rd_req() [metcalf@lcs]
|
|
|
|
| |
Include <zephyr/krb_err.h> if we have Kerberos.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Now logs source address when authentication fails.
|
| |
|
| |
|