[f21/master v2] Avoid the possibility of size variables being unset (#1146585)

Anne Mulhern amulhern at redhat.com
Fri Sep 26 14:12:14 UTC 2014





----- Original Message -----
> From: "Chris Lumens" <clumens at redhat.com>
> To: anaconda-patches at lists.fedorahosted.org
> Sent: Friday, September 26, 2014 9:41:39 AM
> Subject: Re: [f21/master v2] Avoid the possibility of size variables being	unset	(#1146585)
> 
> > The parser in pykickstart defines these values as ints.
> > I think that means that an exception will already have
> > been thrown by parser if they can not be converted to ints.
> 
> Yes, but this is python, so it's entirely possible for someone to come
> along between parse and execute and change the value to some other,
> stupider, type.  You can decide how likely you expect that situation to
> be.  We likely do not worry about it anywhere else, but it probably
> would be possible.
> 
> - Chris
> _______________________________________________
> anaconda-patches mailing list
> anaconda-patches at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/anaconda-patches
> 

In that case what we should really be checking for is the possibility that it is
a Size() (which won't raise a TypeError, but will give us wrong value).
Seems much more likely than somebody setting kickstart size value to a str.

- mulhern


More information about the anaconda-patches mailing list