[PATCH] xf86/modes: Defang stale pointers when copying modes
Keith Packard
keithp at keithp.com
Thu May 31 09:24:40 PDT 2012
Chris Wilson <chris at chris-wilson.co.uk> writes:
> On Wed, 30 May 2012 14:25:40 -0700, Keith Packard <keithp at keithp.com> wrote:
>> Chris Wilson <chris at chris-wilson.co.uk> writes:
>>
>> > +extern _X_EXPORT void
>> > +xf86InternMode(DisplayModePtr intern, const DisplayModeRec * pMode);
>>
>> The name doesn't seem very descriptive to me -- what is 'Intern'
>> supposed to mean?
>
> xf86InternalizeMode? xf86InitModeFromCopy? xf86CopyModeToStatic?
xf86SaveModeContents? (s/Contents/something-more-descriptive/?)
>> What should be done with ->Private? If this mode was ever actually handed
>> to a driver, it might not be NULL...
>
> /* And drop the backend's privates. */
> intern->PrivSize = 0;
> intern->PrivFlags = 0;
> intern->Private = 0;
Yeah, I don't think there's anything more useful to be done here.
--
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20120531/f63a70d6/attachment.pgp>
More information about the xorg-devel
mailing list