diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2012-06-19 05:33:06 -0400 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2012-06-21 04:34:13 -0400 |
| commit | b196a4980ff7bb54db478e2a408dc8b12be15304 (patch) | |
| tree | 22a47e6dd31e0c573df44fb4c75101d69af0bdea | |
| parent | d3decf3a0c1d28c80c76be170373f4c7a7217f09 (diff) | |
drm/edid: don't return stack garbage from supports_rb
We need to initialize this to false, because the is_rb callback only
ever sets it to true.
Noticed while reading through the code.
Cc: stable@vger.kernel.org
Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Reviewed-by: Adam Jackson <ajax@redhat.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
| -rw-r--r-- | drivers/gpu/drm/drm_edid.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index eb92fe257a39..5873e481e5d2 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c | |||
| @@ -610,7 +610,7 @@ static bool | |||
| 610 | drm_monitor_supports_rb(struct edid *edid) | 610 | drm_monitor_supports_rb(struct edid *edid) |
| 611 | { | 611 | { |
| 612 | if (edid->revision >= 4) { | 612 | if (edid->revision >= 4) { |
| 613 | bool ret; | 613 | bool ret = false; |
| 614 | drm_for_each_detailed_block((u8 *)edid, is_rb, &ret); | 614 | drm_for_each_detailed_block((u8 *)edid, is_rb, &ret); |
| 615 | return ret; | 615 | return ret; |
| 616 | } | 616 | } |
