Fix debug CLI node recycling bugs 53/13453/3
authorDave Barach <dbarach@cisco.com>
Thu, 12 Jul 2018 17:00:47 +0000 (13:00 -0400)
committerDave Barach <openvpp@barachs.net>
Thu, 12 Jul 2018 22:30:11 +0000 (22:30 +0000)
commit0610039fd06c760924fb92d0fc7b4d3e0ffeb8e3
tree8b53c47a35898d197318a6e0411ab495c047916a
parent93c90a5c9724518c95b7d5f764becb1f70111ab3
Fix debug CLI node recycling bugs

When creating a new - as opposed to recycled - debug CLI process node,
perform a proper barrier sync and node runtime update. Otherwise, the
graph replicas diverge for some period of time. That's not immediately
fatal, but it's not a good idea, either.

When renaming a debug cli process node, fix all of the name-vector
replicas before freeing the [one-and-only] name vector.

This fixes the so-called stats segment node runtime scraper crash,
which tripped over a replicated dangling reference to the
recently-freed debug CLI node name.

Change-Id: Ieffabd9f003139e534b9d79b88370439907930e5
Signed-off-by: Dave Barach <dbarach@cisco.com>
src/vlib/unix/cli.c