Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 0589be58 authored by Leon Scroggins III's avatar Leon Scroggins III
Browse files

Fix memory leak in AHardwareBuffer_isSupported 

This function creates a new GraphicBuffer to call a single method on it.
Wrap it in an sp<> so that it gets deleted when it's no longer
necessary.

Bug: NA
Test: make and flash
Change-Id: I443015d63245e49e8cf38847030c9da8142cbe50
parent 80cce845

libs/nativewindow/AHardwareBuffer.cpp

+1 −1
Original line number Diff line number Diff line
@@ -370,7 +370,7 @@ int AHardwareBuffer_isSupported(const AHardwareBuffer_Desc* desc) { 
    if (!AHardwareBuffer_isValidDescription(desc, /*log=*/false)) return 0;



    bool supported = false;

    GraphicBuffer* gBuffer = new GraphicBuffer();

    sp<GraphicBuffer> gBuffer(new GraphicBuffer());

    status_t err = gBuffer->isSupported(desc->width, desc->height, desc->format, desc->layers,

                                        desc->usage, &supported);