Changeset: 3a9afb7a8637 for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=3a9afb7a8637
Modified Files:
        gdk/gdk_bbp.c
Branch: Dec2016
Log Message:

Test for NULL pointer before dereferencing.
I've seen crashes during exit because of this, although I don't yet
understand how we can get in the situation.


diffs (19 lines):

diff --git a/gdk/gdk_bbp.c b/gdk/gdk_bbp.c
--- a/gdk/gdk_bbp.c
+++ b/gdk/gdk_bbp.c
@@ -1739,11 +1739,13 @@ BBPexit(void)
                                                bat tp = VIEWtparent(b);
                                                bat vtp = VIEWvtparent(b);
                                                if (tp) {
-                                                       
BBP_cache(tp)->batSharecnt--;
+                                                       if (BBP_desc(tp))
+                                                               
BBP_desc(tp)->batSharecnt--;
                                                        --BBP_lrefs(tp);
                                                }
                                                if (vtp) {
-                                                       
BBP_cache(vtp)->batSharecnt--;
+                                                       if (BBP_desc(vtp))
+                                                               
BBP_desc(vtp)->batSharecnt--;
                                                        --BBP_lrefs(vtp);
                                                }
                                                VIEWdestroy(b);
_______________________________________________
checkin-list mailing list
checkin-list@monetdb.org
https://www.monetdb.org/mailman/listinfo/checkin-list

Reply via email to