Received: from cmr2.ash.ops.us.uu.net (cmr2.ash.ops.us.uu.net [198.5.241.40]) by krbdev.mit.edu (8.9.3p2) with ESMTP id PAA11248; Tue, 6 Jan 2004 15:59:04 -0500 (EST) From: gsu@UU.NET Received: from imr0.ash.ops.us.uu.net by cmr2.ash.ops.us.uu.net with ESMTP (peer crosschecked as: imr0.ash.ops.us.uu.net [153.39.43.11]) id QQpwit16834; Tue, 6 Jan 2004 20:58:57 GMT Received: from imr0.ash.ops.us.uu.net by imr0.ash.ops.us.uu.net with ESMTP (peer crosschecked as: localhost [127.0.0.1]) id QQpwit05965; Tue, 6 Jan 2004 20:58:55 GMT Received: from galaxy.argfrp.us.uu.net by imr0.ash.ops.us.uu.net with ESMTP (peer crosschecked as: galaxy.argfrp.us.uu.net [153.39.56.113]) id QQpwit05957; Tue, 6 Jan 2004 20:58:55 GMT Received: from localhost (gsu@localhost) by galaxy.argfrp.us.uu.net (8.9.3p2/8.9.3) with ESMTP id QQpwit29317; Tue, 6 Jan 2004 20:58:55 GMT X-Authentication-Warning: galaxy.argfrp.us.uu.net: gsu owned process doing -bs Date: Tue, 6 Jan 2004 15:58:55 -0500 (EST) To: Sam Hartman via RT Cc: gsu@UU.NET, krb5-prs@mit.edu Subject: Re: [krbdev.mit.edu #2106] bug in krb5_cc_remove_cred API? In-Reply-To: Message-Id: References: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII RT-Send-Cc: X-RT-Original-Encoding: us-ascii Content-Length: 886 Returning an error would certainly be much cleaner than a null pointer reference. I was so puzzled that I checked the API guide and my code repeatly. If no implementation is intended, why advertise it in the guide? So there is no way that I can remove any expired credential? On Tue, 6 Jan 2004, Sam Hartman via RT wrote: > 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. > >