Move unimplemented fn reset before ctx cleanup
authorBrenden Blanco <bblanco@gmail.com>
Wed, 10 May 2017 18:32:54 +0000 (11:32 -0700)
committerBrenden Blanco <bblanco@gmail.com>
Wed, 10 May 2017 18:32:54 +0000 (11:32 -0700)
Signed-off-by: Brenden Blanco <bblanco@gmail.com>
src/cc/bpf_module.cc

index ca69a7c0d7ebabf3b4006a79d59480dd5fd08628..4cc8191de792f69f889bbdb37a62cd8d07d3e9ea 100644 (file)
@@ -124,10 +124,6 @@ static StatusTuple unimplemented_snprintf(char *, size_t, const void *) {
 }
 
 BPFModule::~BPFModule() {
-  engine_.reset();
-  rw_engine_.reset();
-  ctx_.reset();
-
   for (auto &v : tables_) {
     v->key_sscanf = unimplemented_sscanf;
     v->leaf_sscanf = unimplemented_sscanf;
@@ -135,6 +131,10 @@ BPFModule::~BPFModule() {
     v->leaf_snprintf = unimplemented_snprintf;
   }
 
+  engine_.reset();
+  rw_engine_.reset();
+  ctx_.reset();
+
   ts_->DeletePrefix(Path({id_}));
 }