Kill the zombies
authorMatthew Jordan <mjordan@digium.com>
Tue, 23 Jul 2013 14:49:53 +0000 (14:49 +0000)
committerMatthew Jordan <mjordan@digium.com>
Tue, 23 Jul 2013 14:49:53 +0000 (14:49 +0000)
In previous versions of Asterisk, the zombies roamed freely,
unchecked and uncontrolled. They ravaged Asterisk systems with
their biting and their nashing and their pointy teeth.

Sometimes, you couldn't even hang them up.

Now, zombies are rare. They still *technically* exist in certain
places, but they are controlled. Kind of like a zombie zoo: you can
see them, but you can't touch them, and they can't touch you.
Bring your kids!

Because zombies are now population controlled with a very short lifespan,
there's no reason to rename the channels to '%s<ZOMBIE>'. The channels
are guaranteed to die off quickly; the rename really is just confusing
at this point.

This patch finally removes the renaming. On the plus side: this made
my life easier in CDRs during call pickup and attended transfers to
an Asterisk application. It will make other folks lives easier as well!

Review: https://reviewboard.astierks.org/r/2690/

(closes issue ASTERISK-21699)
Reported by: Matt Jordan

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@395135 65c4cc65-6c06-0410-ace0-fbb531ad65f3

main/channel.c

index d972982..a1a3730 100644 (file)
@@ -6519,10 +6519,6 @@ void ast_do_masquerade(struct ast_channel *original)
        ast_channel_name_set(clonechan, ast_channel_name(original));
        ast_channel_name_set(original, tmp_name);
 
-       /* Now zombify the clonechan. This gets a real rename event. */
-       snprintf(tmp_name, sizeof(tmp_name), "%s<ZOMBIE>", ast_channel_name(clonechan)); /* quick, hide the brains! */
-       __ast_change_name_nolink(clonechan, tmp_name);
-
        /* Swap the technologies */
        t = ast_channel_tech(original);
        ast_channel_tech_set(original, ast_channel_tech(clonechan));