Re: [uml-devel] [PATCH] um: umid: Use strdup to simplify code
... Nice patch elided ... I have a bit of a problem here - Christophe isn't a member of uml-devel, so Mailman is sitting on his post until it's approved by a list moderator, namely me. So far, so good, except that my uml-devel password stopped working, so I now have no ability to approve this (and throw out the reasonable amount of spam that's also waiting for (non-)approval. If anyone has any suggestions about regaining moderator control of this list, please let me know. Jeff -- Don't Limit Your Business. Reach for the Cloud. GigeNET's Cloud Solutions provide you with the tools and support that you need to offload your IT needs and focus on growing your business. Configured For All Businesses. Start Your Cloud Today. https://www.gigenetcloud.com/ ___ User-mode-linux-devel mailing list User-mode-linux-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
Re: [uml-devel] [PATCH] um: umid: Use strdup to simplify code
Hi Jeff, Am 08.07.2015 um 21:56 schrieb Jeff Dike: ... Nice patch elided ... I have a bit of a problem here - Christophe isn't a member of uml-devel, so Mailman is sitting on his post until it's approved by a list moderator, namely me. So far, so good, except that my uml-devel password stopped working, so I now have no ability to approve this (and throw out the reasonable amount of spam that's also waiting for (non-)approval. If anyone has any suggestions about regaining moderator control of this list, please let me know. I'm project manager on UML's sf.net site. Maybe I can reset it. BTW: I'd like to move away from sf.net soon. Maybe to lists.infradead.org. Is that okay for you? Thanks, //richard -- Don't Limit Your Business. Reach for the Cloud. GigeNET's Cloud Solutions provide you with the tools and support that you need to offload your IT needs and focus on growing your business. Configured For All Businesses. Start Your Cloud Today. https://www.gigenetcloud.com/ ___ User-mode-linux-devel mailing list User-mode-linux-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
[uml-devel] [PATCH] um: umid: Use strdup to simplify code
Replace a malloc+strcpy by an equivalent strdup in order to improve readability. Turn a some spaces into a tab to be consistent with the rest of the code. Signed-off-by: Christophe JAILLET christophe.jail...@wanadoo.fr --- arch/um/os-Linux/umid.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/arch/um/os-Linux/umid.c b/arch/um/os-Linux/umid.c index c1dc892..cc9ac2e 100644 --- a/arch/um/os-Linux/umid.c +++ b/arch/um/os-Linux/umid.c @@ -48,15 +48,14 @@ static int __init make_uml_dir(void) strlcat(dir, /, sizeof(dir)); err = -ENOMEM; - uml_dir = malloc(strlen(dir) + 1); + uml_dir = strdup(dir); if (uml_dir == NULL) { - printf(make_uml_dir : malloc failed, errno = %d\n, errno); + printf(make_uml_dir : strdup failed, errno = %d\n, errno); goto err; } - strcpy(uml_dir, dir); if ((mkdir(uml_dir, 0777) 0) (errno != EEXIST)) { - printf(Failed to mkdir '%s': %s\n, uml_dir, strerror(errno)); + printf(Failed to mkdir '%s': %s\n, uml_dir, strerror(errno)); err = -errno; goto err_free; } -- 2.1.4 -- Don't Limit Your Business. Reach for the Cloud. GigeNET's Cloud Solutions provide you with the tools and support that you need to offload your IT needs and focus on growing your business. Configured For All Businesses. Start Your Cloud Today. https://www.gigenetcloud.com/ ___ User-mode-linux-devel mailing list User-mode-linux-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
Re: [uml-devel] [PATCH] um: umid: Use strdup to simplify code
On Wed, Jul 08, 2015 at 10:01:23PM +0200, Richard Weinberger wrote: BTW: I'd like to move away from sf.net soon. Maybe to lists.infradead.org. Is that okay for you? Fine by me - it's your call. Jeff -- Don't Limit Your Business. Reach for the Cloud. GigeNET's Cloud Solutions provide you with the tools and support that you need to offload your IT needs and focus on growing your business. Configured For All Businesses. Start Your Cloud Today. https://www.gigenetcloud.com/ ___ User-mode-linux-devel mailing list User-mode-linux-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
Re: [uml-devel] [PATCH] um: umid: Use strdup to simplify code
Am 08.07.2015 um 21:15 schrieb Christophe JAILLET: Replace a malloc+strcpy by an equivalent strdup in order to improve readability. Makes sense! Turn a some spaces into a tab to be consistent with the rest of the code. Do this in an extra patch. One logical change per patch please. Thanks, //richard -- Don't Limit Your Business. Reach for the Cloud. GigeNET's Cloud Solutions provide you with the tools and support that you need to offload your IT needs and focus on growing your business. Configured For All Businesses. Start Your Cloud Today. https://www.gigenetcloud.com/ ___ User-mode-linux-devel mailing list User-mode-linux-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel