[19472] in Kerberos_V5_Development

home help back first fref pref prev next nref lref last post

Re: Making discarded-qualifiers a gcc error instead of warning?

daemon@ATHENA.MIT.EDU (Will Fiveash)
Thu Sep 1 20:23:20 2016

Date: Thu, 1 Sep 2016 19:23:00 -0500
From: Will Fiveash <will.fiveash@oracle.com>
To: krbdev@mit.edu
Message-ID: <20160902002300.GA15711@oracle.com>
Mail-Followup-To: krbdev@mit.edu
MIME-Version: 1.0
Content-Disposition: inline
In-Reply-To: <20160901225615.GA14221@oracle.com>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: krbdev-bounces@mit.edu

On Thu, Sep 01, 2016 at 05:56:15PM -0500, Will Fiveash wrote:
> I'm in the process of making the Solaris compile of MIT krb more strict
> and one thing I've noticed in the MIT build environment is that
> -Wdiscarded-qualifiers is being treated by gcc as a warning, not an
> error.  Given there are only three of these types of warnings in the
> entire 1.14.3 code base I was wondering if y'all would consider making
> discarded-qualifiers an error and adding a cast to the three lines in
> the source files that need it?

Related to this, the current MIT gcc settings allows:

#include <stdio.h>
#include <stdlib.h>

void
foo(char *arg)
{
	printf("arg is %p\n", arg);
	return;
}

int
main (int argc, char *argv[])
{
	char *bad;

	foo(&bad);  /* <----- this should just be bad */
}

to compile without a hard error (a -Wincompatible-pointer-types warning
is issued).  Shouldn't this bug also cause gcc to error out?

-- 
Will Fiveash
Oracle Solaris Software Engineer
_______________________________________________
krbdev mailing list             krbdev@mit.edu
https://mailman.mit.edu/mailman/listinfo/krbdev

home help back first fref pref prev next nref lref last post