Mesa (18.2): swr: Remove unnecessary memset call

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Thu Aug 9 00:30:42 UTC 2018


Module: Mesa
Branch: 18.2
Commit: 2e903df72fcc54b03c113e7a52ac33034dfecf86
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=2e903df72fcc54b03c113e7a52ac33034dfecf86

Author: Vlad Golovkin <vlad.golovkin.mail at gmail.com>
Date:   Wed Aug  1 20:56:23 2018 +0300

swr: Remove unnecessary memset call

Zeroing memory after calloc is not necessary. This also allows to avoid
possible crash when allocation fails, because memset is called before
checking screen for NULL.

Fixes: a29d63ecf71546c4798c6 "swr: refactor swr_create_screen to allow
                              for proper cleanup on error"
Reviewed-by: Eric Engestrom <eric.engestrom at intel.com>
(cherry picked from commit 9d3a2394e4d3f91f1e23a971ce34aaf031a1e777)

---

 src/gallium/drivers/swr/swr_screen.cpp | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/gallium/drivers/swr/swr_screen.cpp b/src/gallium/drivers/swr/swr_screen.cpp
index fa232b6838..67085444f8 100644
--- a/src/gallium/drivers/swr/swr_screen.cpp
+++ b/src/gallium/drivers/swr/swr_screen.cpp
@@ -1148,7 +1148,6 @@ struct pipe_screen *
 swr_create_screen_internal(struct sw_winsys *winsys)
 {
    struct swr_screen *screen = CALLOC_STRUCT(swr_screen);
-   memset(screen, 0, sizeof(struct swr_screen));
 
    if (!screen)
       return NULL;




More information about the mesa-commit mailing list