rename some xsenstore library functions
PATCH 1/1 rename some xsenstore library functions
Some functions of xenstore library dont have xs_ as prefix. This patch fixes the problem.
# diffstat xenstore3.patch fake_libxc.c | 2 +- xenstored_core.c | 18 +++++++++--------- xenstored_test.h | 2 +- xs.c | 14 +++++++------- xs_lib.c | 8 ++++---- xs_lib.h | 8 ++++---- xs_random.c | 8 ++++---- xs_test.c | 4 ++-- 8 files changed, 32 insertions(+), 32 deletions(-)
===== tools/xenstore/fake_libxc.c 1.1 vs edited =====
tools/xenstore/fake_libxc.c +1 -1
@@ -71,7 +71,7 @@return fd;memset (page,0 ,sizeof (page)); -if (!write_all (fd, page,sizeof (page))) +if (!xs_write_all (fd, page,sizeof (page)))barf_perror ("Failed to write /tmp/xcmap page" );return fd;
===== tools/xenstore/xenstored_core.c 1.1 vs edited =====
tools/xenstore/xenstored_core.c +9 -9
@@ -81,7 +81,7 @@ errno =ENOSPC ;return false; } -return write_all (fd, contents, len); +return xs_write_all (fd, contents, len); }int test_mkdir (const char *dir,int perms);
@@ -443,9 +443,9 @@if (!strings)return NULL ; - *num =count_strings (strings, size); + *num =xs_count_strings (strings, size); ret = talloc_array(node, struct xs_permissions, *num); -if (!strings_to_perms (ret, *num, strings)) +if (!xs_strings_to_perms (ret, *num, strings))corrupt (NULL ,"Permissions corrupt for %s" , node);return ret;
@@ -460,7 +460,7 @@char buffer[MAX_STRLEN (domid_t) +1 ];for (*len =0 , i =0 ; i < num; i++) { -if (!perm_to_string (&perms[i], buffer)) +if (!xs_perm_to_string (&perms[i], buffer))return NULL ; strings = talloc_realloc(node, strings, char,
@@ -506,7 +506,7 @@if (!fd)return NULL ;talloc_set_destructor (tmppath, destroy_path); -if (!write_all (*fd, contents, len)) +if (!xs_write_all (*fd, contents, len))return NULL ;return tmppath;
@@ -721,14 +721,14 @@ permstr =perms_to_strings (dir, &perms,1 , &len); fd =talloc_open (node_permfile (conn->transaction, node),O_WRONLY |O_CREAT |O_EXCL ,0640 ); -if (!fd || !write_all (*fd, permstr, len)) +if (!fd || !xs_write_all (*fd, permstr, len))return false;if (data) {char *datapath =node_datafile (conn->transaction, node); fd =talloc_open (datapath,O_WRONLY |O_CREAT |O_EXCL ,0640 ); -if (!fd || !write_all (*fd, data, datalen)) +if (!fd || !xs_write_all (*fd, data, datalen))return false; }
@@ -878,7 +878,7 @@char *node;struct xs_permissions *perms; - num =count_strings (in->buffer, in->used); + num =xs_count_strings (in->buffer, in->used);if (num <2 )return send_error (conn,EINVAL );
@@ -898,7 +898,7 @@return send_error (conn, errno); perms = talloc_array(node, struct xs_permissions, num); -if (!strings_to_perms (perms, num, in->buffer)) +if (!xs_strings_to_perms (perms, num, in->buffer))return send_error (conn, errno);if (!set_perms (conn->transaction, node, perms, num))
===== tools/xenstore/xenstored_test.h 1.1 vs edited =====
tools/xenstore/xenstored_test.h +1 -1
@@ -21,7 +21,7 @@#ifdef TESTING bool test_write_all (int fd,void *contents,unsigned int len); -#define write_all test_write_all +#define xs_write_all test_write_allint test_mkdir (const char *dir,int perms);#define mkdir test_mkdir
===== tools/xenstore/xs.c 1.1 vs edited =====
tools/xenstore/xs.c +7 -7
@@ -118,7 +118,7 @@#ifdef XSTEST #define read_all read_all_choice -#define write_all write_all_choice +#define xs_write_all write_all_choice#endif static int get_error (const char *errorstring)
@@ -179,11 +179,11 @@ ignorepipe.sa_flags =0 ;sigaction (SIGPIPE , &ignorepipe, &oldact); -if (!write_all (h->fd, &msg,sizeof (msg))) +if (!xs_write_all (h->fd, &msg,sizeof (msg))) goto fail;for (i =0 ; i < num_vecs; i++) -if (!write_all (h->fd, iovec[i].iov_base, iovec[i].iov_len)) +if (!xs_write_all (h->fd, iovec[i].iov_base, iovec[i].iov_len)) goto fail;/* Watches can have fired before reply comes: daemon detects
@@ -253,7 +253,7 @@return NULL ;/* Count the strings. */ - *num =count_strings (strings, len); + *num =xs_count_strings (strings, len);/* Transfer to one big alloc for easy freeing. */ ret =malloc (*num *sizeof (char *) + len);
@@ -342,7 +342,7 @@return NULL ;/* Count the strings: each one perms then domid. */ - *num =count_strings (strings, len); + *num =xs_count_strings (strings, len);/* Transfer to one big alloc for easy freeing. */ ret =malloc (*num *sizeof (struct xs_permissions ));
@@ -351,7 +351,7 @@return NULL ; } -if (!strings_to_perms (ret, *num, strings)) { +if (!xs_strings_to_perms (ret, *num, strings)) {free_no_errno (ret); ret =NULL ; }
@@ -376,7 +376,7 @@for (i =0 ; i < num_perms; i++) {char buffer[MAX_STRLEN (domid_t)+1 ]; -if (!perm_to_string (&perms[i], buffer)) +if (!xs_perm_to_string (&perms[i], buffer)) goto unwind; iov[i+1 ].iov_base =strdup (buffer);
===== tools/xenstore/xs_lib.c 1.1 vs edited =====
tools/xenstore/xs_lib.c +4 -4
@@ -48,7 +48,7 @@ }/* Simple routines for writing to sockets, etc. */ -bool write_all (int fd,const void *data,unsigned int len) +bool xs_write_all (int fd,const void *data,unsigned int len) {while (len) {int done;
@@ -66,7 +66,7 @@ }/* Convert strings to permissions. False if a problem. */ -bool strings_to_perms (struct xs_permissions *perms,unsigned int num, +bool xs_strings_to_perms (struct xs_permissions *perms,unsigned int num,const char *strings) {const char *p;
@@ -104,7 +104,7 @@ }/* Convert permissions to a string (up to len MAX_STRLEN(domid_t)+1). */ -bool perm_to_string (const struct xs_permissions *perm,char *buffer) +bool xs_perm_to_string (const struct xs_permissions *perm,char *buffer) {switch (perm->perms) {case XS_PERM_WRITE :
@@ -128,7 +128,7 @@ }/* Given a string and a length, count how many strings (nul terms). */ -unsigned int count_strings (const char *strings,unsigned int len) +unsigned int xs_count_strings (const char *strings,unsigned int len) {unsigned int num;const char *p;
===== tools/xenstore/xs_lib.h 1.1 vs edited =====
tools/xenstore/xs_lib.h +4 -4
@@ -48,16 +48,16 @@const char *xs_daemon_transactions (void );/* Simple write function: loops for you. */ -bool write_all (int fd,const void *data,unsigned int len); +bool xs_write_all (int fd,const void *data,unsigned int len);/* Convert strings to permissions. False if a problem. */ -bool strings_to_perms (struct xs_permissions *perms,unsigned int num, +bool xs_strings_to_perms (struct xs_permissions *perms,unsigned int num,const char *strings);/* Convert permissions to a string (up to len MAX_STRLEN(domid_t)+1). */ -bool perm_to_string (const struct xs_permissions *perm,char *buffer); +bool xs_perm_to_string (const struct xs_permissions *perm,char *buffer);/* Given a string and a length, count how many strings (nul terms). */ -unsigned int count_strings (const char *strings,unsigned int len); +unsigned int xs_count_strings (const char *strings,unsigned int len);#endif /* _XS_LIB_H */
===== tools/xenstore/xs_random.c 1.1 vs edited =====
tools/xenstore/xs_random.c +4 -4
@@ -223,10 +223,10 @@release_file (perms, size);return ret; } - *num =count_strings (perms, size); + *num =xs_count_strings (perms, size); ret = new_array(struct xs_permissions, *num); -if (!strings_to_perms (ret, *num, perms)) +if (!xs_strings_to_perms (ret, *num, perms))barf ("Reading permissions from %s" , permfile);release_file (perms, size);return ret;
@@ -267,7 +267,7 @@for (i =0 ; i < num; i++) {char buffer[100 ]; -if (!perm_to_string (&perms[i], buffer)) { +if (!xs_perm_to_string (&perms[i], buffer)) {int saved_errno = errno;close (fd); errno = saved_errno;
@@ -536,7 +536,7 @@ ret =talloc_asprintf_append (ret,"%s%s: " , spacing, dir[i]);for (j =0 ; j < numperms; j++) {char buffer[100 ]; -if (!perm_to_string (&perms[j], buffer)) +if (!xs_perm_to_string (&perms[j], buffer))barf ("perm to string" ); ret =talloc_asprintf_append (ret,"%s " , buffer); }
===== tools/xenstore/xs_test.c 1.1 vs edited =====
tools/xenstore/xs_test.c +2 -2
@@ -153,7 +153,7 @@ {if (fd ==-2 )return write_all_shmem (fd, data, len); -return write_all (fd, data, len); +return xs_write_all (fd, data, len); }/* We want access to internal functions. */
@@ -491,7 +491,7 @@printf ("%s%s: " , spacing, dir[i]);for (j =0 ; j < numperms; j++) {char buffer[100 ]; -if (!perm_to_string (&perms[j], buffer)) +if (!xs_perm_to_string (&perms[j], buffer))barf ("perm to string" );printf ("%s " , buffer); }
_______________________________________________
Xen-devel mailing list
[email protected]
http://lists.xensource.com/xen-devel
Quoted 2 lines · aq
Some functions of xenstore library dont have
xs_as prefix. This patch fixes the problem.
But it's not a problem, actually. Only public functions need to start with xs_.
Please don't apply this patch.
Rusty.
Signature · 1 line
A bad analogy is like a leaky screwdriver -- Richard Braakman
Quoted 6 lines · Rusty Russell
Quoted 2 lines · aq
Some functions of xenstore library dont have
xs_as prefix. This patch fixes the problem.But it's not a problem, actually. Only public functions need to start with
xs_.
i see the point, but the reason of this patch is that i am worried that the code (which uses this lib) might accidentally has a conflict function.
otherwise it is OK.
regards,
aq