From 83789acac046c93170f00c1c8fc2c1452ff315a1 Mon Sep 17 00:00:00 2001 From: Matt Wilson Date: Fri, 6 Sep 2013 12:52:05 -0700 Subject: 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 Cc: Stefano Stabellini Acked-By: Samuel Thibault --- extras/mini-os/pcifront.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'extras/mini-os/pcifront.c') diff --git a/extras/mini-os/pcifront.c b/extras/mini-os/pcifront.c index f9ae768170..e6a5f4e7bf 100644 --- a/extras/mini-os/pcifront.c +++ b/extras/mini-os/pcifront.c @@ -149,7 +149,7 @@ struct pcifront_dev *init_pcifront(char *_nodename) struct pcifront_dev *dev; - char path[strlen(nodename) + 1 + 10 + 1]; + char path[strlen(nodename) + strlen("/backend-id") + 1]; if (!_nodename && pcidev) return pcidev; @@ -237,8 +237,8 @@ done: printk("backend at %s\n", dev->backend); { - char path[strlen(dev->backend) + 1 + 5 + 1]; - char frontpath[strlen(nodename) + 1 + 5 + 1]; + char path[strlen(dev->backend) + strlen("/state") + 1]; + char frontpath[strlen(nodename) + strlen("/state") + 1]; XenbusState state; snprintf(path, sizeof(path), "%s/state", dev->backend); @@ -322,7 +322,7 @@ void shutdown_pcifront(struct pcifront_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("/event-channel") + 1]; printk("close pci: backend at %s\n",dev->backend); -- cgit v1.2.3