Received: from konishi-polis.mit.edu (HOTASS-9.MIT.EDU [18.101.1.46]) by krbdev.mit.edu (8.9.3p2) with ESMTP id PAA11212; Tue, 6 Jan 2004 15:24:00 -0500 (EST)
Received: by konishi-polis.mit.edu (Postfix, from userid 8042) id D45051515EA; Tue,  6 Jan 2004 15:23:55 -0500 (EST)
To: rt@krbdev.mit.edu
Cc: 
Subject: Re: [krbdev.mit.edu #2106] bug in krb5_cc_remove_cred API?
References: <rt-2106-9064.1.14201087485483@krbdev.mit.edu>
From: Sam Hartman <hartmans@mit.edu>
Date: Tue, 06 Jan 2004 15:23:55 -0500
In-Reply-To: <rt-2106-9064.1.14201087485483@krbdev.mit.edu> (rt-comment@krbdev.mit.edu's message of "Tue, 6 Jan 2004 14:33:29 -0500 (EST)")
Message-Id: <tslptdwddwk.fsf@konishi-polis.mit.edu>
User-Agent: Gnus/5.1002 (Gnus v5.10.2) Emacs/21.2 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
RT-Send-Cc: 
X-RT-Original-Encoding: us-ascii
Content-Length: 530

Well, it is not really clear to me if this is a bug or missing code.
We certainly don't implement remove_cred, so there is no way your test
program could successfully remove a credential.

Clearly we should eventually implement that code and its absence is a
bug.

Having the unimplemented API segfault definitely does draw the missing
functionality to one's attention.  It's true that an assertion would
be cleaner than a null pointer dereference.
The alternative would be to return an error indicating the API is
unimplemented.