git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 1/2] Move add_to_string to "quote.c" and make it extern.

From
Christian Couder <chriscool@tuxfamily.org>
Date
Sep 11, 2006, 04:59 UTC
Message-ID
<20060911065922.93139e5d.chriscool@tuxfamily.org>
In-Reply-To
<200609110643.13486.chriscool@tuxfamily.org>
So that this function may be used in places other than "rsh.c".
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
---
 quote.c |   29 +++++++++++++++++++++++++++++
 quote.h |    6 ++++++
 rsh.c   |   29 -----------------------------
 3 files changed, 35 insertions(+), 29 deletions(-)
diff --git a/quote.c b/quote.c
index a38786c..200e4e2 100644
--- a/quote.c
+++ b/quote.c
@@ -106,6 +106,35 @@ char *sq_quote_argv(const char** argv, i
 	return buf;
 }
 
+/*
+ * Append a string to a string buffer, with or without shell quoting.
+ * Return true if the buffer overflowed.
+ */
+int add_to_string(char **ptrp, int *sizep, const char *str, int quote)
+{
+	char *p = *ptrp;
+	int size = *sizep;
+	int oc;
+	int err = 0;
+
+	if ( quote ) {
+		oc = sq_quote_buf(p, size, str);
+	} else {
+		oc = strlen(str);
+		memcpy(p, str, (oc >= size) ? size-1 : oc);
+	}
+
+	if ( oc >= size ) {
+		err = 1;
+		oc = size-1;
+	}
+
+	*ptrp  += oc;
+	**ptrp  = '\0';
+	*sizep -= oc;
+	return err;
+}
+
 char *sq_dequote(char *arg)
 {
 	char *dst = arg;
diff --git a/quote.h b/quote.h
index a6c4611..1a29e79 100644
--- a/quote.h
+++ b/quote.h
@@ -33,6 +33,12 @@ extern void sq_quote_print(FILE *stream,
 extern size_t sq_quote_buf(char *dst, size_t n, const char *src);
 extern char *sq_quote_argv(const char** argv, int count);
 
+/*
+ * Append a string to a string buffer, with or without shell quoting.
+ * Return true if the buffer overflowed.
+ */
+extern int add_to_string(char **ptrp, int *sizep, const char *str, int quote);
+
 /* This unwraps what sq_quote() produces in place, but returns
  * NULL if the input does not look like what sq_quote would have
  * produced.
diff --git a/rsh.c b/rsh.c
index 07166ad..8a1db45 100644
--- a/rsh.c
+++ b/rsh.c
@@ -8,35 +8,6 @@ #include "cache.h"
 
 #define COMMAND_SIZE 4096
 
-/*
- * Append a string to a string buffer, with or without shell quoting.
- * Return true if the buffer overflowed.
- */
-static int add_to_string(char **ptrp, int *sizep, const char *str, int quote)
-{
-	char *p = *ptrp;
-	int size = *sizep;
-	int oc;
-	int err = 0;
-
-	if ( quote ) {
-		oc = sq_quote_buf(p, size, str);
-	} else {
-		oc = strlen(str);
-		memcpy(p, str, (oc >= size) ? size-1 : oc);
-	}
-
-	if ( oc >= size ) {
-		err = 1;
-		oc = size-1;
-	}
-
-	*ptrp  += oc;
-	**ptrp  = '\0';
-	*sizep -= oc;
-	return err;
-}
-
 int setup_connection(int *fd_in, int *fd_out, const char *remote_prog, 
 		     char *url, int rmt_argc, char **rmt_argv)
 {
-- 
1.4.2.g430e
Previous: Christian CouderNext: Christian Couder
Message 4 of 6 in “Fix memory leak in "connect.c".”
  1. Fix memory leak in "connect.c".Christian Couder, Sep 7, 2006
  2. Junio C HamanoSep 7, 2006
  3. 0/2 Command preparation cleanup in "connect.c" (was: [PATCH] Fix memory leak in "connect.c".)Christian Couder, Sep 11, 2006
  4. 1/2 Move add_to_string to "quote.c" and make it extern.Christian Couder, Sep 11, 2006
  5. 2/2 Fix a memory leak in "connect.c" and die if command too long.Christian Couder, Sep 11, 2006
  6. Junio C HamanoSep 11, 2006

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.