Re: [PATCH v2] drm: use ida to allocate connector ids

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Aug 07, 2013 at 04:28:35AM -0400, Ilia Mirkin wrote:
> On Wed, Aug 7, 2013 at 4:00 AM, Ville Syrjälä
> <ville.syrjala@xxxxxxxxxxxxxxx> wrote:
> > On Tue, Jul 30, 2013 at 03:51:49AM -0400, Ilia Mirkin wrote:
> >> @@ -722,8 +741,9 @@ int drm_connector_init(struct drm_device *dev,
> >>       connector->dev = dev;
> >>       connector->funcs = funcs;
> >>       connector->connector_type = connector_type;
> >> -     connector->connector_type_id =
> >> -             ++drm_connector_enum_list[connector_type].count; /* TODO */
> >> +     ret = ida_get_new_above(count, 1, &connector->connector_type_id);
> >> +     if (ret)
> >> +             goto out;
> >
> > Leak.
> 
> I assume this is in reference to the drm_mode_object_get above? I just
> noticed that... or was there something else too?

Just the obj id.

-- 
Ville Syrjälä
Intel OTC
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel





[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux