- Subject: Re: [Ethereal-dev] Re: [Ethereal-cvs] rev 17504: /trunk/gtk/: main.c
- From: Ulf Lamping <ulf.lamping@xxxxxx>
- Date: Tue, 07 Mar 2006 23:31:03 +0100
Gerald Combs wrote:
It was ostensibly to validate the input of a function. In reality it
fixed a problem Coverity wasn't complaining about. The real problem was
that it may have been possible to pass a NULL value to
get_interface_descriptive_name(). Fixed in r17513.
That sounds much better :-)
However, some recent "coverity problem fixes" sounded to me fixing
similiar problems the "same way" ...
Reminds me of compiler warnings going away with a cast, without
understanding the problem behind it ...
Regards, ULFL