[PATCH 6/7] Add wpa_supplicant_fork command.
Christian Mauderer
christian.mauderer at embedded-brains.de
Thu Aug 17 08:22:19 UTC 2017
That's a great function for debugging but I'm not sure whether it would
be good as a command available to all. There should be at least some big
warning that it can't be started multiple times. In the ideal case, add
a lock so that it can only be started once.
A little more into the future, it most likely should be replaced by a
deamon similar to the dhcpcd. Or it should be started from the rc.conf
parser.
Am 17.08.2017 um 03:33 schrieb Sichen Zhao:
> Add fork command for wpa supplicant
> to start a new task.
> ---
> freebsd/contrib/wpa/wpa_supplicant/main.c | 67 +++++++++++++++++++++-
> libbsd.py | 1 +
> libbsd_waf.py | 1 +
> rtemsbsd/include/machine/rtems-bsd-commands.h | 2 +
> rtemsbsd/include/rtems/netcmds-config.h | 2 +
> .../rtems/rtems-bsd-shell-wpa_supplicant_fork.c | 36 ++++++++++++
> 6 files changed, 108 insertions(+), 1 deletion(-)
> create mode 100644 rtemsbsd/rtems/rtems-bsd-shell-wpa_supplicant_fork.c
>
> diff --git a/freebsd/contrib/wpa/wpa_supplicant/main.c b/freebsd/contrib/wpa/wpa_supplicant/main.c
> index 7347951..b2af31e 100644
> --- a/freebsd/contrib/wpa/wpa_supplicant/main.c
> +++ b/freebsd/contrib/wpa/wpa_supplicant/main.c
> @@ -18,7 +18,9 @@
> #include "wpa_supplicant_i.h"
> #include "driver_i.h"
> #include "p2p_supplicant.h"
> -
Again: Minus is bad.
> +#ifdef __rtems__
> +#include <assert.h>
> +#endif /* __rtems__ */
>
> static void usage(void)
> {
> @@ -168,6 +170,69 @@ int rtems_bsd_command_wpa_supplicant(int argc, char **argv)
>
> return exit_code;
> }
> +
> +struct myparams {
> + int argc;
> + char ** argv;
Use BSD-Style indentations. There are some more places in that file with
wrong indentation. Please fix them too.
> +};
> +
> +static void
> +new_wpa_supplicant_task(rtems_task_argument arg)
> +{
> + int argc;
> + char ** argv;
> + int i;
> +
> + struct myparams *params = (struct myparams *)arg;
> + argc = params->argc;
> + argv = params->argv;
> +
> + rtems_bsd_command_wpa_supplicant(argc, argv);
> +
> + for (i = 0; i < params->argc; i++) {
> + free(params->argv[i]);
> + }
> + free(params->argv);
> + free(params);
> +
> + rtems_task_delete( RTEMS_SELF );
> +}
> +
> +int rtems_bsd_command_wpa_supplicant_fork(int argc, char **argv)
> +{
> + rtems_status_code sc;
> + rtems_id id;
> + int i;
> +
> + struct myparams *params = malloc(sizeof(struct myparams));
> + if (params == NULL)
> + return NULL;
> +
> + params->argc = argc;
> + params->argv = malloc((argc + 1) * sizeof(argv[0]));
> + if (params->argv == NULL)
> + return NULL;
> +
> + for (i = 0; i < argc; i++) {
> + params->argv[i] = strdup(argv[i]);
> + if (params->argv[i] == NULL)
You should free params->argv[x] for all x < i in that case. After that
you should also free params->argv. Then you can return. Otherwise this
might be a memory leak.
> + return NULL;
> + }
> + params->argv[argc] = NULL;
> +
> + sc = rtems_task_create(
> + rtems_build_name('W', 'P', 'A', 'S'),
> + RTEMS_MAXIMUM_PRIORITY - 1,
> + 8 * RTEMS_MINIMUM_STACK_SIZE,
> + RTEMS_DEFAULT_MODES,
> + RTEMS_FLOATING_POINT,
> + &id
> + );
> + assert(sc == RTEMS_SUCCESSFUL);
> +
> + sc = rtems_task_start(id, new_wpa_supplicant_task, params);
> + assert(sc == RTEMS_SUCCESSFUL);
> +}
> #endif /* __rtems__ */
>
> int main(int argc, char *argv[])
> diff --git a/libbsd.py b/libbsd.py
> index a0b7f1a..9bb6f0b 100644
> --- a/libbsd.py
> +++ b/libbsd.py
> @@ -4237,6 +4237,7 @@ def usr_sbin_wpa_supplicant(mm):
> mod.addRTEMSSourceFiles(
> [
> 'rtems/rtems-bsd-shell-wpa_supplicant.c',
> + 'rtems/rtems-bsd-shell-wpa_supplicant_fork.c',
> ],
> mm.generator['source']()
> )
> diff --git a/libbsd_waf.py b/libbsd_waf.py
> index 02b2cbd..28e3cf0 100644
> --- a/libbsd_waf.py
> +++ b/libbsd_waf.py
> @@ -2288,6 +2288,7 @@ def build(bld):
> 'rtemsbsd/rtems/rtems-bsd-shell-vmstat.c',
> 'rtemsbsd/rtems/rtems-bsd-shell-wlanstats.c',
> 'rtemsbsd/rtems/rtems-bsd-shell-wpa_supplicant.c',
> + 'rtemsbsd/rtems/rtems-bsd-shell-wpa_supplicant_fork.c',
> 'rtemsbsd/rtems/rtems-bsd-syscall-api.c',
> 'rtemsbsd/rtems/rtems-kernel-assert.c',
> 'rtemsbsd/rtems/rtems-kernel-autoconf.c',
> diff --git a/rtemsbsd/include/machine/rtems-bsd-commands.h b/rtemsbsd/include/machine/rtems-bsd-commands.h
> index 03a09bc..32aba44 100644
> --- a/rtemsbsd/include/machine/rtems-bsd-commands.h
> +++ b/rtemsbsd/include/machine/rtems-bsd-commands.h
> @@ -64,6 +64,8 @@ int rtems_bsd_command_dhcpcd(int argc, char **argv);
>
> int rtems_bsd_command_wpa_supplicant(int argc, char **argv);
>
> +int rtems_bsd_command_wpa_supplicant_fork(int argc, char **argv);
> +
> int rtems_bsd_command_tcpdump(int argc, char **argv);
>
> int rtems_bsd_command_sysctl(int argc, char **argv);
> diff --git a/rtemsbsd/include/rtems/netcmds-config.h b/rtemsbsd/include/rtems/netcmds-config.h
> index 046c859..4af4323 100644
> --- a/rtemsbsd/include/rtems/netcmds-config.h
> +++ b/rtemsbsd/include/rtems/netcmds-config.h
> @@ -41,6 +41,8 @@ extern rtems_shell_cmd_t rtems_shell_TCPDUMP_Command;
>
> extern rtems_shell_cmd_t rtems_shell_WPA_SUPPLICANT_Command;
>
> +extern rtems_shell_cmd_t rtems_shell_WPA_SUPPLICANT_FORK_Command;
> +
> extern rtems_shell_cmd_t rtems_shell_SYSCTL_Command;
>
> extern rtems_shell_cmd_t rtems_shell_VMSTAT_Command;
> diff --git a/rtemsbsd/rtems/rtems-bsd-shell-wpa_supplicant_fork.c b/rtemsbsd/rtems/rtems-bsd-shell-wpa_supplicant_fork.c
> new file mode 100644
> index 0000000..046b611
> --- /dev/null
> +++ b/rtemsbsd/rtems/rtems-bsd-shell-wpa_supplicant_fork.c
> @@ -0,0 +1,36 @@
> +/*
> + * Copyright (c) 2017 Sichen Zhao. All rights reserved.
> + *
> + * <zsc19940506 at gmail.com>
> + *
> + * Redistribution and use in source and binary forms, with or without
> + * modification, are permitted provided that the following conditions
> + * are met:
> + * 1. Redistributions of source code must retain the above copyright
> + * notice, this list of conditions and the following disclaimer.
> + * 2. Redistributions in binary form must reproduce the above copyright
> + * notice, this list of conditions and the following disclaimer in the
> + * documentation and/or other materials provided with the distribution.
> + *
> + * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
> + * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
> + * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
> + * ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
> + * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
> + * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
> + * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
> + * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
> + * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
> + * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
> + * SUCH DAMAGE.
> + */
> +
> +#include <rtems/netcmds-config.h>
> +#include <machine/rtems-bsd-commands.h>
> +
> +rtems_shell_cmd_t rtems_shell_WPA_SUPPLICANT_FORK_Command = {
> + .name = "wpa_supplicant_fork",
> + .usage = "wpa_supplicant_fork [args]",
Like already said on top: Add at least a BIG FAT WARNING that the
process can not be restarted with that command. wpa_supplicant has to
terminate on itself before you can restart it again.
> + .topic = "net",
> + .command = rtems_bsd_command_wpa_supplicant_fork
> +};
>
--
--------------------------------------------
embedded brains GmbH
Christian Mauderer
Dornierstr. 4
D-82178 Puchheim
Germany
email: christian.mauderer at embedded-brains.de
Phone: +49-89-18 94 741 - 18
Fax: +49-89-18 94 741 - 08
PGP: Public key available on request.
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.
More information about the devel
mailing list