[Asterisk-code-review] app_nbscat: Remove deprecated module. (asterisk[19])

Joshua Colp asteriskteam at digium.com
Mon Aug 16 12:55:25 CDT 2021


Joshua Colp has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/16295 )


Change subject: app_nbscat: Remove deprecated module.
......................................................................

app_nbscat: Remove deprecated module.

ASTERISK-29590

Change-Id: I87cf0f536b77d222c8eda003376ac47fae86ed43
---
D apps/app_nbscat.c
A doc/UPGRADE-staging/app_nbscat_removal.txt
2 files changed, 6 insertions(+), 225 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/95/16295/1

diff --git a/apps/app_nbscat.c b/apps/app_nbscat.c
deleted file mode 100644
index c714cbb..0000000
--- a/apps/app_nbscat.c
+++ /dev/null
@@ -1,225 +0,0 @@
-/*
- * Asterisk -- An open source telephony toolkit.
- *
- * Copyright (C) 1999 - 2005, Digium, Inc.
- *
- * Mark Spencer <markster at digium.com>
- *
- * See http://www.asterisk.org for more information about
- * the Asterisk project. Please do not directly contact
- * any of the maintainers of this project for assistance;
- * the project provides a web site, mailing lists and IRC
- * channels for your use.
- *
- * This program is free software, distributed under the terms of
- * the GNU General Public License Version 2. See the LICENSE file
- * at the top of the source tree.
- */
-
-/*! \file
- *
- * \brief Silly application to play an NBScat file -- uses nbscat8k
- *
- * \author Mark Spencer <markster at digium.com>
- *
- * \ingroup applications
- */
-
-/*** MODULEINFO
-	<support_level>deprecated</support_level>
-	<deprecated_in>16</deprecated_in>
-	<removed_in>19</removed_in>
- ***/
-
-#include "asterisk.h"
-
-#include <fcntl.h>
-#include <sys/time.h>
-#include <sys/socket.h>
-#include <signal.h>
-
-#include "asterisk/lock.h"
-#include "asterisk/file.h"
-#include "asterisk/channel.h"
-#include "asterisk/frame.h"
-#include "asterisk/pbx.h"
-#include "asterisk/module.h"
-#include "asterisk/translate.h"
-#include "asterisk/app.h"
-#include "asterisk/format_cache.h"
-
-/*** DOCUMENTATION
-	<application name="NBScat" language="en_US">
-		<synopsis>
-			Play an NBS local stream.
-		</synopsis>
-		<syntax />
-		<description>
-			<para>Executes nbscat to listen to the local NBS stream.
-			User can exit by pressing any key.</para>
-		</description>
-	</application>
- ***/
-
-#define LOCAL_NBSCAT "/usr/local/bin/nbscat8k"
-#define NBSCAT "/usr/bin/nbscat8k"
-
-#ifndef AF_LOCAL
-#define AF_LOCAL AF_UNIX
-#endif
-
-static char *app = "NBScat";
-
-static int NBScatplay(int fd)
-{
-	int res;
-
-	res = ast_safe_fork(0);
-	if (res < 0) {
-		ast_log(LOG_WARNING, "Fork failed\n");
-	}
-
-	if (res) {
-		return res;
-	}
-
-	if (ast_opt_high_priority)
-		ast_set_priority(0);
-
-	dup2(fd, STDOUT_FILENO);
-	ast_close_fds_above_n(STDERR_FILENO);
-	/* Most commonly installed in /usr/local/bin */
-	execl(NBSCAT, "nbscat8k", "-d", (char *)NULL);
-	execl(LOCAL_NBSCAT, "nbscat8k", "-d", (char *)NULL);
-	fprintf(stderr, "Execute of nbscat8k failed\n");
-	_exit(0);
-}
-
-static int timed_read(int fd, void *data, int datalen)
-{
-	int res;
-	struct pollfd fds[1];
-	fds[0].fd = fd;
-	fds[0].events = POLLIN;
-	res = ast_poll(fds, 1, 2000);
-	if (res < 1) {
-		ast_log(LOG_NOTICE, "Selected timed out/errored out with %d\n", res);
-		return -1;
-	}
-	return read(fd, data, datalen);
-
-}
-
-static int NBScat_exec(struct ast_channel *chan, const char *data)
-{
-	int res=0;
-	int fds[2];
-	int ms = -1;
-	int pid = -1;
-	struct ast_format *owriteformat;
-	struct timeval next;
-	struct ast_frame *f;
-	struct myframe {
-		struct ast_frame f;
-		char offset[AST_FRIENDLY_OFFSET];
-		short frdata[160];
-	} myf;
-
-	if (socketpair(AF_LOCAL, SOCK_STREAM, 0, fds)) {
-		ast_log(LOG_WARNING, "Unable to create socketpair\n");
-		return -1;
-	}
-
-	ast_stopstream(chan);
-
-	owriteformat = ao2_bump(ast_channel_writeformat(chan));
-	res = ast_set_write_format(chan, ast_format_slin);
-	if (res < 0) {
-		ast_log(LOG_WARNING, "Unable to set write format to signed linear\n");
-		ao2_cleanup(owriteformat);
-		return -1;
-	}
-
-	myf.f.frametype = AST_FRAME_VOICE;
-	myf.f.subclass.format = ast_format_slin;
-	myf.f.mallocd = 0;
-	myf.f.offset = AST_FRIENDLY_OFFSET;
-	myf.f.src = __PRETTY_FUNCTION__;
-	myf.f.delivery.tv_sec = 0;
-	myf.f.delivery.tv_usec = 0;
-	myf.f.data.ptr = myf.frdata;
-
-	res = NBScatplay(fds[1]);
-	/* Wait 1000 ms first */
-	next = ast_tvnow();
-	next.tv_sec += 1;
-	if (res >= 0) {
-		pid = res;
-		/* Order is important -- there's almost always going to be mp3...  we want to prioritize the
-		   user */
-		for (;;) {
-			ms = ast_tvdiff_ms(next, ast_tvnow());
-			if (ms <= 0) {
-				res = timed_read(fds[0], myf.frdata, sizeof(myf.frdata));
-				if (res > 0) {
-					myf.f.datalen = res;
-					myf.f.samples = res / 2;
-					if (ast_write(chan, &myf.f) < 0) {
-						res = -1;
-						break;
-					}
-				} else {
-					ast_debug(1, "No more mp3\n");
-					res = 0;
-					break;
-				}
-				next = ast_tvadd(next, ast_samp2tv(myf.f.samples, 8000));
-			} else {
-				ms = ast_waitfor(chan, ms);
-				if (ms < 0) {
-					ast_debug(1, "Hangup detected\n");
-					res = -1;
-					break;
-				}
-				if (ms) {
-					f = ast_read(chan);
-					if (!f) {
-						ast_debug(1, "Null frame == hangup() detected\n");
-						res = -1;
-						break;
-					}
-					if (f->frametype == AST_FRAME_DTMF) {
-						ast_debug(1, "User pressed a key\n");
-						ast_frfree(f);
-						res = 0;
-						break;
-					}
-					ast_frfree(f);
-				}
-			}
-		}
-	}
-	close(fds[0]);
-	close(fds[1]);
-	ast_frfree(&myf.f);
-
-	if (pid > -1)
-		kill(pid, SIGKILL);
-	if (!res && owriteformat)
-		ast_set_write_format(chan, owriteformat);
-	ao2_cleanup(owriteformat);
-
-	return res;
-}
-
-static int unload_module(void)
-{
-	return ast_unregister_application(app);
-}
-
-static int load_module(void)
-{
-	return ast_register_application_xml(app, NBScat_exec);
-}
-
-AST_MODULE_INFO_STANDARD_DEPRECATED(ASTERISK_GPL_KEY, "Silly NBS Stream Application");
diff --git a/doc/UPGRADE-staging/app_nbscat_removal.txt b/doc/UPGRADE-staging/app_nbscat_removal.txt
new file mode 100644
index 0000000..a1d15d8
--- /dev/null
+++ b/doc/UPGRADE-staging/app_nbscat_removal.txt
@@ -0,0 +1,6 @@
+Subject: app_nbscat
+Master-Only: True
+
+This module was deprecated in Asterisk 16
+and is now being removed in accordance with
+the Asterisk Module Deprecation policy.

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

Gerrit-Project: asterisk
Gerrit-Branch: 19
Gerrit-Change-Id: I87cf0f536b77d222c8eda003376ac47fae86ed43
Gerrit-Change-Number: 16295
Gerrit-PatchSet: 1
Gerrit-Owner: Joshua Colp <jcolp at sangoma.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210816/72f46c2d/attachment-0001.html>


More information about the asterisk-code-review mailing list