[Asterisk-code-review] Testsuite: Add extra memory debugging to CDR blind transfer ... (testsuite[master])

[Anonymous Coward]

Anonymous Coward #1000019 has submitted this change and it was merged.

Change subject: Testsuite: Add extra memory debugging to CDR blind transfer test.
......................................................................


Testsuite: Add extra memory debugging to CDR blind transfer test.

The CDR blind transfer test has on rare occasions failed because of
detected memory corruption. We can see that the memory that is corrupted
is part of a JSON object allocation. Aside from that, it's impossible to
see who is allocating that JSON object. There are way too many possible
places that this memory may be corrupted.

I also cannot reproduce this problem locally. So what I propose here is
to add memory backtraces to the CDR blind transfer test. This way, if
the automatic testsuite runs have the memory corruption again, we can
get a full trace of where the allocation happened. This can provide a
narrower scope for where to be looking for the potential heap
corruption.

Change-Id: I9185549df7b35b88f17b30f5ba2034f2d42c6dc5
---
A tests/cdr/cdr_properties/blind-transfer-accountcode/configs/ast1/cli.conf
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Scott Griepentrog: Looks good to me, but someone else must approve
  Anonymous Coward #1000019: Verified
  Joshua Colp: Looks good to me, approved



diff --git a/tests/cdr/cdr_properties/blind-transfer-accountcode/configs/ast1/cli.conf b/tests/cdr/cdr_properties/blind-transfer-accountcode/configs/ast1/cli.conf
new file mode 100644
index 0000000..51fa561
--- /dev/null
+++ b/tests/cdr/cdr_properties/blind-transfer-accountcode/configs/ast1/cli.conf
@@ -0,0 +1,2 @@
+[startup_commands]
+memory backtrace on = yes

-- 
To view, visit https://gerrit.asterisk.org/3038
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9185549df7b35b88f17b30f5ba2034f2d42c6dc5
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Scott Griepentrog <sgriepentrog at digium.com>