[inferbo] Always update summary

Summary:
In some rare cases, the CFG is broken, the analysis cannot reach the exit node and we won't update the summary for this procedure.
In this diff, the summary is gonna be updated with `Bottom` instead.

Reviewed By: skcho

Differential Revision: D13801731

fbshipit-source-id: 79d412429

infer/src/bufferoverrun/bufferOverrunChecker.ml

@@ -659,14 +659,12 @@ let compute_invariant_map_and_check : Callbacks.proc_callback_args -> invariant_
       Report.check_proc summary proc_desc tenv integer_type_widths cfg inv_map
       |> Report.report_errors summary |> Report.forget_locs locals |> Report.for_summary
     in
-    let exit_mem = extract_post (CFG.Node.id exit_node) inv_map in
+    let exit_mem =
-    match exit_mem with
+      extract_post (CFG.Node.id exit_node) inv_map
-    | Some exit_mem ->
+      |> Option.value ~default:Bottom |> Dom.Mem.forget_locs locals |> Dom.Mem.unset_oenv
-        let exit_mem = exit_mem |> Dom.Mem.forget_locs locals |> Dom.Mem.unset_oenv in
+    in
     let payload = (exit_mem, cond_set) in
     Payload.update_summary payload summary
-    | _ ->
-        summary
   in
   NodePrinter.finish_session underlying_exit_node ;
   if Config.hoisting_report_only_expensive then