mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH 1/1] sanbox: add linux_execve and linux_exec command
@ 2011-10-23  9:59 Jean-Christophe PLAGNIOL-VILLARD
  2011-11-22  7:52 ` Jean-Christophe PLAGNIOL-VILLARD
  2011-11-22 20:07 ` Sascha Hauer
  0 siblings, 2 replies; 3+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2011-10-23  9:59 UTC (permalink / raw)
  To: barebox

this will allow to execute a program of the host from barebox

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
---
 arch/sandbox/mach-sandbox/include/mach/linux.h |    2 +
 arch/sandbox/os/common.c                       |   22 +++++++++
 commands/Kconfig                               |    4 ++
 commands/Makefile                              |    1 +
 commands/linux_exec.c                          |   58 ++++++++++++++++++++++++
 5 files changed, 87 insertions(+), 0 deletions(-)
 create mode 100644 commands/linux_exec.c

diff --git a/arch/sandbox/mach-sandbox/include/mach/linux.h b/arch/sandbox/mach-sandbox/include/mach/linux.h
index 7c2386d..5917fe9 100644
--- a/arch/sandbox/mach-sandbox/include/mach/linux.h
+++ b/arch/sandbox/mach-sandbox/include/mach/linux.h
@@ -10,6 +10,8 @@ ssize_t linux_write(int fd, const void *buf, size_t count);
 off_t linux_lseek(int fildes, off_t offset);
 int linux_tstc(int fd);
 
+int linux_execve(const char * filename, char *const argv[], char *const envp[]);
+
 int barebox_register_console(char *name_template, int stdinfd, int stdoutfd);
 
 struct linux_console_data {
diff --git a/arch/sandbox/os/common.c b/arch/sandbox/os/common.c
index 5074a06..0833959 100644
--- a/arch/sandbox/os/common.c
+++ b/arch/sandbox/os/common.c
@@ -45,6 +45,7 @@
 #include <errno.h>
 #include <signal.h>
 #include <sys/select.h>
+#include <sys/wait.h>
 /*
  * ...except the ones needed to connect with barebox
  */
@@ -193,6 +194,27 @@ off_t linux_lseek(int fd, off_t offset)
 	return lseek(fd, offset, SEEK_SET);
 }
 
+int linux_execve(const char * filename, char *const argv[], char *const envp[])
+{
+	pid_t pid, tpid;
+	int execve_status;
+
+	pid = fork();
+
+	if (pid == -1) {
+		perror("linux_execve");
+		return pid;
+	} else if (pid == 0) {
+		exit(execve(filename, argv, envp));
+	} else {
+		do {
+			tpid = wait(&execve_status);
+		} while(tpid != pid);
+
+		return execve_status;
+	}
+}
+
 extern void start_barebox(void);
 extern void mem_malloc_init(void *start, void *end);
 
diff --git a/commands/Kconfig b/commands/Kconfig
index 11386f9..1eae25a 100644
--- a/commands/Kconfig
+++ b/commands/Kconfig
@@ -124,6 +124,10 @@ config CMD_TIME
 	  checking for ctrl-c, so the time command can be used with commands
 	  which are interruptible with ctrl-c.
 
+config CMD_LINUX_EXEC
+	bool "linux exec"
+	depends on LINUX
+
 endmenu
 
 menu "file commands                 "
diff --git a/commands/Makefile b/commands/Makefile
index d50ca68..f50ace6 100644
--- a/commands/Makefile
+++ b/commands/Makefile
@@ -59,3 +59,4 @@ obj-$(CONFIG_CMD_USB)		+= usb.o
 obj-$(CONFIG_CMD_TIME)		+= time.o
 obj-$(CONFIG_CMD_BOOT_CONFIG)	+= boot_config.o
 obj-$(CONFIG_CMD_BOOT_MENU)	+= boot_menu.o
+obj-$(CONFIG_CMD_LINUX_EXEC)	+= linux_exec.o
diff --git a/commands/linux_exec.c b/commands/linux_exec.c
new file mode 100644
index 0000000..e147629
--- /dev/null
+++ b/commands/linux_exec.c
@@ -0,0 +1,58 @@
+/*
+ * Copyright (c) 2011 Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2
+ * as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+ */
+
+#include <common.h>
+#include <command.h>
+#include <mach/linux.h>
+
+static int do_linux_exec(struct command *cmdtp, int argc, char *argv[])
+{
+	int ret;
+	char **newargv;
+	char *newenv[] = { NULL };
+	int i, j;
+
+	if (argc < 2)
+		return 1;
+
+	newargv = xzalloc(sizeof(char*) * argc);
+
+	for (j = 0, i = 1; i < argc; i++, j++)
+		newargv[j] = argv[i];
+
+	newargv[j] = NULL;
+	ret = linux_execve(argv[1], newargv, newenv);
+
+	if (ret)
+		return 1;
+
+	return 0;
+}
+
+BAREBOX_CMD_HELP_START(linux_exec)
+BAREBOX_CMD_HELP_USAGE("linux_exec ...\n")
+BAREBOX_CMD_HELP_SHORT("Execute a command on the host\n")
+BAREBOX_CMD_HELP_END
+
+BAREBOX_CMD_START(linux_exec)
+	.cmd		= do_linux_exec,
+	.usage		= "Execute a command on the host",
+	BAREBOX_CMD_HELP(cmd_linux_exec_help)
+BAREBOX_CMD_END
-- 
1.7.7


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/1] sanbox: add linux_execve and linux_exec command
  2011-10-23  9:59 [PATCH 1/1] sanbox: add linux_execve and linux_exec command Jean-Christophe PLAGNIOL-VILLARD
@ 2011-11-22  7:52 ` Jean-Christophe PLAGNIOL-VILLARD
  2011-11-22 20:07 ` Sascha Hauer
  1 sibling, 0 replies; 3+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2011-11-22  7:52 UTC (permalink / raw)
  To: barebox

On 17:59 Sun 23 Oct     , Jean-Christophe PLAGNIOL-VILLARD wrote:
> this will allow to execute a program of the host from barebox
> 
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> ---
ping

Best Regards,
J.
>  arch/sandbox/mach-sandbox/include/mach/linux.h |    2 +
>  arch/sandbox/os/common.c                       |   22 +++++++++
>  commands/Kconfig                               |    4 ++
>  commands/Makefile                              |    1 +
>  commands/linux_exec.c                          |   58 ++++++++++++++++++++++++
>  5 files changed, 87 insertions(+), 0 deletions(-)
>  create mode 100644 commands/linux_exec.c
> 
> diff --git a/arch/sandbox/mach-sandbox/include/mach/linux.h b/arch/sandbox/mach-sandbox/include/mach/linux.h
> index 7c2386d..5917fe9 100644
> --- a/arch/sandbox/mach-sandbox/include/mach/linux.h
> +++ b/arch/sandbox/mach-sandbox/include/mach/linux.h
> @@ -10,6 +10,8 @@ ssize_t linux_write(int fd, const void *buf, size_t count);
>  off_t linux_lseek(int fildes, off_t offset);
>  int linux_tstc(int fd);
>  
> +int linux_execve(const char * filename, char *const argv[], char *const envp[]);
> +
>  int barebox_register_console(char *name_template, int stdinfd, int stdoutfd);
>  
>  struct linux_console_data {
> diff --git a/arch/sandbox/os/common.c b/arch/sandbox/os/common.c
> index 5074a06..0833959 100644
> --- a/arch/sandbox/os/common.c
> +++ b/arch/sandbox/os/common.c
> @@ -45,6 +45,7 @@
>  #include <errno.h>
>  #include <signal.h>
>  #include <sys/select.h>
> +#include <sys/wait.h>
>  /*
>   * ...except the ones needed to connect with barebox
>   */
> @@ -193,6 +194,27 @@ off_t linux_lseek(int fd, off_t offset)
>  	return lseek(fd, offset, SEEK_SET);
>  }
>  
> +int linux_execve(const char * filename, char *const argv[], char *const envp[])
> +{
> +	pid_t pid, tpid;
> +	int execve_status;
> +
> +	pid = fork();
> +
> +	if (pid == -1) {
> +		perror("linux_execve");
> +		return pid;
> +	} else if (pid == 0) {
> +		exit(execve(filename, argv, envp));
> +	} else {
> +		do {
> +			tpid = wait(&execve_status);
> +		} while(tpid != pid);
> +
> +		return execve_status;
> +	}
> +}
> +
>  extern void start_barebox(void);
>  extern void mem_malloc_init(void *start, void *end);
>  
> diff --git a/commands/Kconfig b/commands/Kconfig
> index 11386f9..1eae25a 100644
> --- a/commands/Kconfig
> +++ b/commands/Kconfig
> @@ -124,6 +124,10 @@ config CMD_TIME
>  	  checking for ctrl-c, so the time command can be used with commands
>  	  which are interruptible with ctrl-c.
>  
> +config CMD_LINUX_EXEC
> +	bool "linux exec"
> +	depends on LINUX
> +
>  endmenu
>  
>  menu "file commands                 "
> diff --git a/commands/Makefile b/commands/Makefile
> index d50ca68..f50ace6 100644
> --- a/commands/Makefile
> +++ b/commands/Makefile
> @@ -59,3 +59,4 @@ obj-$(CONFIG_CMD_USB)		+= usb.o
>  obj-$(CONFIG_CMD_TIME)		+= time.o
>  obj-$(CONFIG_CMD_BOOT_CONFIG)	+= boot_config.o
>  obj-$(CONFIG_CMD_BOOT_MENU)	+= boot_menu.o
> +obj-$(CONFIG_CMD_LINUX_EXEC)	+= linux_exec.o
> diff --git a/commands/linux_exec.c b/commands/linux_exec.c
> new file mode 100644
> index 0000000..e147629
> --- /dev/null
> +++ b/commands/linux_exec.c
> @@ -0,0 +1,58 @@
> +/*
> + * Copyright (c) 2011 Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2
> + * as published by the Free Software Foundation.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
> + */
> +
> +#include <common.h>
> +#include <command.h>
> +#include <mach/linux.h>
> +
> +static int do_linux_exec(struct command *cmdtp, int argc, char *argv[])
> +{
> +	int ret;
> +	char **newargv;
> +	char *newenv[] = { NULL };
> +	int i, j;
> +
> +	if (argc < 2)
> +		return 1;
> +
> +	newargv = xzalloc(sizeof(char*) * argc);
> +
> +	for (j = 0, i = 1; i < argc; i++, j++)
> +		newargv[j] = argv[i];
> +
> +	newargv[j] = NULL;
> +	ret = linux_execve(argv[1], newargv, newenv);
> +
> +	if (ret)
> +		return 1;
> +
> +	return 0;
> +}
> +
> +BAREBOX_CMD_HELP_START(linux_exec)
> +BAREBOX_CMD_HELP_USAGE("linux_exec ...\n")
> +BAREBOX_CMD_HELP_SHORT("Execute a command on the host\n")
> +BAREBOX_CMD_HELP_END
> +
> +BAREBOX_CMD_START(linux_exec)
> +	.cmd		= do_linux_exec,
> +	.usage		= "Execute a command on the host",
> +	BAREBOX_CMD_HELP(cmd_linux_exec_help)
> +BAREBOX_CMD_END
> -- 
> 1.7.7

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/1] sanbox: add linux_execve and linux_exec command
  2011-10-23  9:59 [PATCH 1/1] sanbox: add linux_execve and linux_exec command Jean-Christophe PLAGNIOL-VILLARD
  2011-11-22  7:52 ` Jean-Christophe PLAGNIOL-VILLARD
@ 2011-11-22 20:07 ` Sascha Hauer
  1 sibling, 0 replies; 3+ messages in thread
From: Sascha Hauer @ 2011-11-22 20:07 UTC (permalink / raw)
  To: Jean-Christophe PLAGNIOL-VILLARD; +Cc: barebox

Hi Jean Christophe,

On Sun, Oct 23, 2011 at 05:59:21PM +0800, Jean-Christophe PLAGNIOL-VILLARD wrote:
> this will allow to execute a program of the host from barebox
> +
> +#include <common.h>
> +#include <command.h>
> +#include <mach/linux.h>
> +
> +static int do_linux_exec(struct command *cmdtp, int argc, char *argv[])
> +{
> +	int ret;
> +	char **newargv;
> +	char *newenv[] = { NULL };
> +	int i, j;
> +
> +	if (argc < 2)
> +		return 1;
> +
> +	newargv = xzalloc(sizeof(char*) * argc);
> +
> +	for (j = 0, i = 1; i < argc; i++, j++)
> +		newargv[j] = argv[i];
> +
> +	newargv[j] = NULL;
> +	ret = linux_execve(argv[1], newargv, newenv);
> +
> +	if (ret)
> +		return 1;
> +
> +	return 0;

You are loosing memory here.

Sascha

> +}
> +
> +BAREBOX_CMD_HELP_START(linux_exec)
> +BAREBOX_CMD_HELP_USAGE("linux_exec ...\n")
> +BAREBOX_CMD_HELP_SHORT("Execute a command on the host\n")
> +BAREBOX_CMD_HELP_END
> +
> +BAREBOX_CMD_START(linux_exec)
> +	.cmd		= do_linux_exec,
> +	.usage		= "Execute a command on the host",
> +	BAREBOX_CMD_HELP(cmd_linux_exec_help)
> +BAREBOX_CMD_END
> -- 
> 1.7.7
> 
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-11-22 20:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-10-23  9:59 [PATCH 1/1] sanbox: add linux_execve and linux_exec command Jean-Christophe PLAGNIOL-VILLARD
2011-11-22  7:52 ` Jean-Christophe PLAGNIOL-VILLARD
2011-11-22 20:07 ` Sascha Hauer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox