Messages in this thread | | | From | Alex Deucher <> | Date | Mon, 29 Apr 2019 15:29:14 -0400 | Subject | Re: [PATCH][next] drm/amd/display: fix incorrect null check on pointer |
| |
On Fri, Apr 26, 2019 at 5:48 PM Colin King <colin.king@canonical.com> wrote: > > From: Colin Ian King <colin.king@canonical.com> > > Currently an allocation is being made but the allocation failure > check is being performed on another pointer. Fix this by checking > the correct pointer. Also use the normal kernel idiom for null > pointer checks. > > Addresses-Coverity: ("Resource leak") > Fixes: 43e3ac8389ef ("drm/amd/display: Add function to copy DC streams") > Signed-off-by: Colin Ian King <colin.king@canonical.com>
Applied. Thanks!
Alex
> --- > drivers/gpu/drm/amd/display/dc/core/dc_stream.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_stream.c b/drivers/gpu/drm/amd/display/dc/core/dc_stream.c > index 6200df3edcd0..96e97d25d639 100644 > --- a/drivers/gpu/drm/amd/display/dc/core/dc_stream.c > +++ b/drivers/gpu/drm/amd/display/dc/core/dc_stream.c > @@ -168,7 +168,7 @@ struct dc_stream_state *dc_copy_stream(const struct dc_stream_state *stream) > struct dc_stream_state *new_stream; > > new_stream = kzalloc(sizeof(struct dc_stream_state), GFP_KERNEL); > - if (stream == NULL) > + if (!new_stream) > return NULL; > > memcpy(new_stream, stream, sizeof(struct dc_stream_state)); > -- > 2.20.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel
| |