[anaconda][rhel7-branch/master][PATCH] Don't crash if escrow certificate is requested without network access (#1085265)
Anne Mulhern
amulhern at redhat.com
Fri Sep 26 15:57:11 UTC 2014
----- Original Message -----
> From: "Chris Lumens" <clumens at redhat.com>
> To: anaconda-patches at lists.fedorahosted.org
> Sent: Friday, September 26, 2014 11:39:34 AM
> Subject: Re: [anaconda][rhel7-branch/master][PATCH] Don't crash if escrow certificate is requested without network
> access (#1085265)
>
> > There's nothing wrong with the value, it's the context, so KickstartError
> > is the most appropriate to raise.
>
> """
> It also exports several exception classes:
>
> KickstartError - A generic exception class.
> """
>
> So, KicsktartError is really intended for things that don't fit anywhere
> else. I see one use in pykickstart/commands/repo.py that really ought to
> be something else, and then there's a couple in pykickstart/parser.py
> that at least need to be audited.
>
> But in general, it's meant as a top-level exception class for everything
> to derive from. It's also more meant to be caught than raised. Think
> about "except Exception" instead of "raise Exception".
>
> - Chris
> _______________________________________________
> anaconda-patches mailing list
> anaconda-patches at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/anaconda-patches
>
Unfortunately, our only options are KSValueError, KSVersionError, KSParseError
and the situation of no network being available definitely does not match any of them.
Maybe a KSRuntimeError is needed?
In any case catching KickstartError in the two places I mentioned
will fix the bug and be correct as well, AFAICT.
Those places where KickstartError is raised can be
audited at leisure and replaced w/ a more appropriate subclass
whenever it becomes available.
- mulhern
More information about the anaconda-patches
mailing list