diff options
author | Matt Wilson <msw@amazon.com> | 2013-09-06 12:52:05 -0700 |
---|---|---|
committer | Ian Campbell <ian.campbell@citrix.com> | 2013-09-10 10:55:05 +0100 |
commit | 83789acac046c93170f00c1c8fc2c1452ff315a1 (patch) | |
tree | 2a8e1fb9f8f5f2d775b1444b24b4ab564f2241d2 /extras/mini-os/fbfront.c | |
parent | f949cbbeda5dea85810de9eee7613add709e9e91 (diff) | |
download | xen-83789acac046c93170f00c1c8fc2c1452ff315a1.tar.gz xen-83789acac046c93170f00c1c8fc2c1452ff315a1.tar.bz2 xen-83789acac046c93170f00c1c8fc2c1452ff315a1.zip |
minios: clean up allocation of char arrays used for xenbus paths
This patch cleans up instances of char array allocation where string
lengths were manually counted to use strlen() instead. There are no
functional changes in this patch.
Signed-off-by: Matt Wilson <msw@amazon.com>
Cc: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Acked-By: Samuel Thibault <samuel.thibault@ens-lyon.org>
Diffstat (limited to 'extras/mini-os/fbfront.c')
-rw-r--r-- | extras/mini-os/fbfront.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/extras/mini-os/fbfront.c b/extras/mini-os/fbfront.c index 6eddb3cca3..9148496953 100644 --- a/extras/mini-os/fbfront.c +++ b/extras/mini-os/fbfront.c @@ -75,7 +75,7 @@ struct kbdfront_dev *init_kbdfront(char *_nodename, int abs_pointer) char* nodename = _nodename ? _nodename : "device/vkbd/0"; struct kbdfront_dev *dev; - char path[strlen(nodename) + 1 + 10 + 1]; + char path[strlen(nodename) + strlen("/backend-id") + 1]; printk("******************* KBDFRONT for %s **********\n\n\n", nodename); @@ -239,7 +239,7 @@ void shutdown_kbdfront(struct kbdfront_dev *dev) char* err = NULL; XenbusState state; - char path[strlen(dev->backend) + 1 + 5 + 1]; + char path[strlen(dev->backend) + strlen("/state") + 1]; char nodename[strlen(dev->nodename) + strlen("/request-abs-pointer") + 1]; printk("close kbd: backend at %s\n",dev->backend); @@ -413,7 +413,7 @@ struct fbfront_dev *init_fbfront(char *_nodename, unsigned long *mfns, int width unsigned long mapped; char* nodename = _nodename ? _nodename : "device/vfb/0"; - char path[strlen(nodename) + 1 + 10 + 1]; + char path[strlen(nodename) + strlen("/backend-id") + 1]; printk("******************* FBFRONT for %s **********\n\n\n", nodename); @@ -520,7 +520,7 @@ done: { XenbusState state; - char path[strlen(dev->backend) + 1 + 14 + 1]; + char path[strlen(dev->backend) + strlen("/request-update") + 1]; char frontpath[strlen(nodename) + strlen("/state") + 1]; snprintf(path, sizeof(path), "%s/state", dev->backend); @@ -631,7 +631,7 @@ void shutdown_fbfront(struct fbfront_dev *dev) char* err = NULL; XenbusState state; - char path[strlen(dev->backend) + 1 + 5 + 1]; + char path[strlen(dev->backend) + strlen("/state") + 1]; char nodename[strlen(dev->nodename) + strlen("/feature-update") + 1]; printk("close fb: backend at %s\n",dev->backend); |