[Asterisk-code-review] ast_coredumper: Fix issues with naming (asterisk[13])

George Joseph asteriskteam at digium.com
Fri Aug 28 09:40:10 CDT 2020


George Joseph has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/14881 )


Change subject: ast_coredumper: Fix issues with naming
......................................................................

ast_coredumper: Fix issues with naming

If you run ast_coredumper --tarball-coredumps in the same directory
as the actual coredump, tar can fail because the link to the
actual coredump becomes recursive.  The resulting tarball will
have everything _except_ the coredump (which is usually what
you need)

There's also an issue that the directory name in the tarball
is the same as the coredump so if you extract the tarball the
directory it creates will overwrite the coredump.

So:

 * Made the link to the coredump use the absolute path to the
   file instead of a relative one.  This prevents the recursive
   link and allows tar to add the coredump.

 * The tarballed directory is now named <coredump>.output instead
   of just <coredump> so if you expand the tarball it won't
   overwrite the coredump.

Change-Id: I8b3eeb26e09a577c702ff966924bb0a2f9a759ea
---
M contrib/scripts/ast_coredumper
1 file changed, 2 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/81/14881/1

diff --git a/contrib/scripts/ast_coredumper b/contrib/scripts/ast_coredumper
index ee62ab8..9d9f8bc 100755
--- a/contrib/scripts/ast_coredumper
+++ b/contrib/scripts/ast_coredumper
@@ -478,7 +478,7 @@
 
 # Now iterate over the coredumps and dump the debugging info
 for i in ${!COREDUMPS[@]} ; do
-	cf=${COREDUMPS[$i]}
+	cf=$(readlink -ne ${COREDUMPS[$i]})
 	echo "Processing $cf"
 
 	cfdir=`dirname ${cf}`
@@ -520,7 +520,7 @@
 		cp -a /${libdir}/asterisk/* ${dest}/${libdir}/asterisk/
 		cp -a /usr/sbin/asterisk ${dest}/usr/sbin
 		rm -rf ${tf}
-		tar -chzf ${tf} --transform="s/^[.]/${cfname}/" -C ${dest} .
+		tar -chzf ${tf} --transform="s/^[.]/${cfname}.output/" -C ${dest} .
 		sleep 3
 		rm -rf ${dest}
 		echo "Created $tf"

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/14881
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Change-Id: I8b3eeb26e09a577c702ff966924bb0a2f9a759ea
Gerrit-Change-Number: 14881
Gerrit-PatchSet: 1
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20200828/ee859c7d/attachment.html>


More information about the asterisk-code-review mailing list