[sheepdog] Duplicated line at lib/net.c:519

Roy Yang bsdnet at outlook.com
Fri Mar 13 06:04:03 CET 2015


While browsing the source of sheepdog,  seems those two lines are the same:

memcpy(bytes + 12, &sin->sin_addr, 4);
memcpy(bytes + 12, &sin->sin_addr, 4); 

Attached patch is to remove the duplicated line if this is a minor bug.

>From 4f5fc977ea12e2048fc025b66fbbeea9c411157d Mon Sep 17 00:00:00 2001
From: Roy Yang <ryang at ryang-dt.(none)>
Date: Thu, 12 Mar 2015 21:39:00 -0700
Subject: [PATCH] Remove the duplicated code line

Signed-off-by: Roy Yang <bsdnet at outlook.com>
---
 lib/net.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/lib/net.c b/lib/net.c
index 5b34a1c..6e84937 100644
--- a/lib/net.c
+++ b/lib/net.c
@@ -519,7 +519,6 @@ int get_local_addr(uint8_t *bytes)
                        sin = (struct sockaddr_in *)ifa->ifa_addr;
                        memset(bytes, 0, 12);
                        memcpy(bytes + 12, &sin->sin_addr, 4);
-                       memcpy(bytes + 12, &sin->sin_addr, 4);
                        sd_notice("found IPv4 address");
                        goto out;
                case AF_INET6:
-- 
1.7.2.3


Cheers, 

Roy
 		 	   		  
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wpkg.org/pipermail/sheepdog/attachments/20150312/ae252530/attachment-0003.html>


More information about the sheepdog mailing list