bgpd: Prevent Null pointer usage

Coverity rightly points out that bgp_table_top might return
NULL and immediately deref'ing it might be a problem.
Add a bit of safety.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

diff --git a/bgpd/bgp_bmp.c b/bgpd/bgp_bmp.c
index db330f998c266a757aabec79e49637269f0746a8..af88547ca94aa2bc5a63a67ffe8be0e5df795181 100644 (file)
--- a/bgpd/bgp_bmp.c
+++ b/bgpd/bgp_bmp.c
@@ -951,8 +951,11 @@ afibreak:
                /* initialize syncrdpos to the first
                 * mid-layer table entry
                 */
-               if (!bmp->syncrdpos)
+               if (!bmp->syncrdpos) {
                        bmp->syncrdpos = bgp_table_top(table);
+                       if (!bmp->syncrdpos)
+                               goto eor;
+               }
 
                /* look for a valid mid-layer table */
                do {