diff options
| author | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2013-12-24 12:58:01 +0100 |
|---|---|---|
| committer | Rob Clark <robdclark@gmail.com> | 2014-03-31 11:06:51 -0400 |
| commit | 06fb220b2d1a80be5621fa06ef276a3890e6be78 (patch) | |
| tree | d4463a5bd48f4e59565f327eb19edf819e59daab | |
| parent | 30b6f8f6792abe582ef25100984a113ac9bc88dd (diff) | |
| download | op-kernel-dev-06fb220b2d1a80be5621fa06ef276a3890e6be78.zip op-kernel-dev-06fb220b2d1a80be5621fa06ef276a3890e6be78.tar.gz | |
drm/omap: Don't dereference list head when the connectors list is empty
The connectors list iterator returns the list head when the list is
empty. Fix it by returning NULL in that case.
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Rob Clark <robdclark@gmail.com>
Signed-off-by: Rob Clark <robdclark@gmail.com>
| -rw-r--r-- | drivers/gpu/drm/omapdrm/omap_fb.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/gpu/drm/omapdrm/omap_fb.c b/drivers/gpu/drm/omapdrm/omap_fb.c index f466c4a..19a9b861 100644 --- a/drivers/gpu/drm/omapdrm/omap_fb.c +++ b/drivers/gpu/drm/omapdrm/omap_fb.c @@ -306,7 +306,8 @@ struct drm_connector *omap_framebuffer_get_next_connector( struct drm_connector *connector = from; if (!from) - return list_first_entry(connector_list, typeof(*from), head); + return list_first_entry_or_null(connector_list, typeof(*from), + head); list_for_each_entry_from(connector, connector_list, head) { if (connector != from) { |
