]> git.proxmox.com Git - mirror_edk2.git/commitdiff
StdLib: Fix GCC warnings/errors caused by variables being set but not used.
authorOlivier Martin <olivier.martin@arm.com>
Thu, 30 Oct 2014 01:05:22 +0000 (01:05 +0000)
committerdarylm503 <darylm503@Edk2>
Thu, 30 Oct 2014 01:05:22 +0000 (01:05 +0000)
Removed variables that had no effect on code behavior.

Fifo.c::FIFO_Dequeue: Replaced instances of "Self->ElementSize" with preexisting variable "SizeOfElement".

IIOutilities.c::IIO_GetInChar: Fixed variable of wrong, but compatible, type and made updating of housekeeping variables dependent upon successful completion of reading from the buffer.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Olivier Martin <olivier.martin@arm.com>
Reviewed by: Daryl McDaniel <daryl.mcdaniel@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@16276 6f19259b-4bc3-4df7-8a09-765794883524

StdLib/BsdSocketLib/getnameinfo.c
StdLib/BsdSocketLib/res_comp.c
StdLib/BsdSocketLib/res_mkupdate.c
StdLib/EfiSocketLib/Ip4.c
StdLib/EfiSocketLib/Tcp4.c
StdLib/EfiSocketLib/Tcp6.c
StdLib/LibC/Containers/Queues/Fifo.c
StdLib/LibC/Uefi/InteractiveIO/IIOutilities.c
StdLib/LibC/Uefi/InteractiveIO/IIOwrite.c
StdLib/LibC/Uefi/InteractiveIO/NonCanonRead.c
StdLib/LibC/Uefi/writev.c

index fab3460dc0762b1a08a1b29dc59544e8e6ae35ca..6d74306880eee0ddd32fd3e9638e3b405e83ac00 100644 (file)
@@ -1,5 +1,15 @@
-/*     $NetBSD: getnameinfo.c,v 1.45 2006/10/15 16:14:46 christos Exp $        */\r
-/*     $KAME: getnameinfo.c,v 1.45 2000/09/25 22:43:56 itojun Exp $    */\r
+/** @file\r
+  Copyright (c) 2010 - 2014, Intel Corporation. All rights reserved.<BR>\r
+  This program and the accompanying materials are licensed and made available\r
+  under the terms and conditions of the BSD License which accompanies this\r
+  distribution.  The full text of the license may be found at\r
+  http://opensource.org/licenses/bsd-license.php.\r
+\r
+  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
+  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
+**/\r
+/*  $NetBSD: getnameinfo.c,v 1.45 2006/10/15 16:14:46 christos Exp $  */\r
+/*  $KAME: getnameinfo.c,v 1.45 2000/09/25 22:43:56 itojun Exp $  */\r
 \r
 /*\r
  * Copyright (c) 2000 Ben Harris.\r
@@ -89,33 +99,33 @@ hexname(
   );\r
 \r
 static const struct afd {\r
-       int             a_af;\r
-       socklen_t       a_addrlen;\r
-       socklen_t       a_socklen;\r
-       int             a_off;\r
+  int   a_af;\r
+  socklen_t a_addrlen;\r
+  socklen_t a_socklen;\r
+  int   a_off;\r
 } afdl [] = {\r
 #ifdef INET6\r
-       {PF_INET6, sizeof(struct in6_addr), sizeof(struct sockaddr_in6),\r
-               offsetof(struct sockaddr_in6, sin6_addr)},\r
+  {PF_INET6, sizeof(struct in6_addr), sizeof(struct sockaddr_in6),\r
+    offsetof(struct sockaddr_in6, sin6_addr)},\r
 #endif\r
-       {PF_INET, sizeof(struct in_addr), sizeof(struct sockaddr_in),\r
-               offsetof(struct sockaddr_in, sin_addr)},\r
-       {0, 0, 0, 0},\r
+  {PF_INET, sizeof(struct in_addr), sizeof(struct sockaddr_in),\r
+    offsetof(struct sockaddr_in, sin_addr)},\r
+  {0, 0, 0, 0},\r
 };\r
 \r
 struct sockinet {\r
-       u_char  si_len;\r
-       u_char  si_family;\r
-       u_short si_port;\r
+  u_char  si_len;\r
+  u_char  si_family;\r
+  u_short si_port;\r
 };\r
 \r
 static int getnameinfo_inet __P((const struct sockaddr *, socklen_t, char *,\r
     socklen_t, char *, socklen_t, int));\r
 #ifdef INET6\r
 static int ip6_parsenumeric __P((const struct sockaddr *, const char *, char *,\r
-                                socklen_t, int));\r
+         socklen_t, int));\r
 static int ip6_sa2str __P((const struct sockaddr_in6 *, char *, size_t,\r
-                                int));\r
+         int));\r
 #endif\r
 static int getnameinfo_link __P((const struct sockaddr *, socklen_t, char *,\r
     socklen_t, char *, socklen_t, int));\r
@@ -137,17 +147,17 @@ getnameinfo(
   )\r
 {\r
 \r
-       switch (sa->sa_family) {\r
-       case AF_INET:\r
-       case AF_INET6:\r
-               return getnameinfo_inet(sa, salen, host, hostlen,\r
-                   serv, servlen, flags);\r
-       case AF_LINK:\r
-               return getnameinfo_link(sa, salen, host, hostlen,\r
-                   serv, servlen, flags);\r
-       default:\r
-               return EAI_FAMILY;\r
-       }\r
+  switch (sa->sa_family) {\r
+  case AF_INET:\r
+  case AF_INET6:\r
+    return getnameinfo_inet(sa, salen, host, hostlen,\r
+        serv, servlen, flags);\r
+  case AF_LINK:\r
+    return getnameinfo_link(sa, salen, host, hostlen,\r
+        serv, servlen, flags);\r
+  default:\r
+    return EAI_FAMILY;\r
+  }\r
 }\r
 \r
 \r
@@ -167,193 +177,193 @@ getnameinfo_inet(
   int flags\r
   )\r
 {\r
-       const struct afd *afd;\r
-       struct servent *sp;\r
-       struct hostent *hp;\r
-       u_short port;\r
-       int family, i;\r
-       const char *addr;\r
-       u_int32_t v4a;\r
-       char numserv[512];\r
-       char numaddr[512];\r
-\r
-       /* sa is checked below */\r
-       /* host may be NULL */\r
-       /* serv may be NULL */\r
-\r
-       if (sa == NULL)\r
-               return EAI_FAIL;\r
+  const struct afd *afd;\r
+  struct servent *sp;\r
+  struct hostent *hp;\r
+  u_short port;\r
+  int family, i;\r
+  const char *addr;\r
+  u_int32_t v4a;\r
+  char numserv[512];\r
+  char numaddr[512];\r
+\r
+  /* sa is checked below */\r
+  /* host may be NULL */\r
+  /* serv may be NULL */\r
+\r
+  if (sa == NULL)\r
+    return EAI_FAIL;\r
 \r
 #ifdef BSD4_4\r
-       if (sa->sa_len != salen)\r
-               return EAI_FAIL;\r
+  if (sa->sa_len != salen)\r
+    return EAI_FAIL;\r
 #endif\r
 \r
-       family = sa->sa_family;\r
-       for (i = 0; afdl[i].a_af; i++)\r
-               if (afdl[i].a_af == family) {\r
-                       afd = &afdl[i];\r
-                       goto found;\r
-               }\r
-       return EAI_FAMILY;\r
+  family = sa->sa_family;\r
+  for (i = 0; afdl[i].a_af; i++)\r
+    if (afdl[i].a_af == family) {\r
+      afd = &afdl[i];\r
+      goto found;\r
+    }\r
+  return EAI_FAMILY;\r
 \r
  found:\r
-       if (salen != afd->a_socklen)\r
-               return EAI_FAIL;\r
-\r
-       /* network byte order */\r
-       port = ((const struct sockinet *)(const void *)sa)->si_port;\r
-       addr = (const char *)(const void *)sa + afd->a_off;\r
-\r
-       if (serv == NULL || servlen == 0) {\r
-               /*\r
-                * do nothing in this case.\r
-                * in case you are wondering if "&&" is more correct than\r
-                * "||" here: rfc2553bis-03 says that serv == NULL OR\r
-                * servlen == 0 means that the caller does not want the result.\r
-                */\r
-       } else {\r
-               if (flags & NI_NUMERICSERV)\r
-                       sp = NULL;\r
-               else {\r
-                       struct servent_data svd;\r
-//                     struct servent sv;\r
-\r
-                       (void)memset(&svd, 0, sizeof(svd));\r
-                       sp = getservbyport_r(port,\r
-                               (flags & NI_DGRAM) ? "udp" : "tcp", &sv, &svd);\r
-                       endservent_r(&svd);\r
-               }\r
-               if (sp) {\r
-                       if (strlen(sp->s_name) + 1 > servlen)\r
-                               return EAI_MEMORY;\r
-                       strlcpy(serv, sp->s_name, servlen);\r
-               } else {\r
-                       snprintf(numserv, sizeof(numserv), "%u", ntohs(port));\r
-                       if (strlen(numserv) + 1 > servlen)\r
-                               return EAI_MEMORY;\r
-                       strlcpy(serv, numserv, servlen);\r
-               }\r
-       }\r
-\r
-       switch (sa->sa_family) {\r
-       case AF_INET:\r
-               v4a = (u_int32_t)\r
-                   ntohl(((const struct sockaddr_in *)\r
-                   (const void *)sa)->sin_addr.s_addr);\r
-               if (IN_MULTICAST(v4a) || IN_EXPERIMENTAL(v4a))\r
-                       flags |= NI_NUMERICHOST;\r
-               v4a >>= IN_CLASSA_NSHIFT;\r
-               if (v4a == 0)\r
-                       flags |= NI_NUMERICHOST;\r
-               break;\r
+  if (salen != afd->a_socklen)\r
+    return EAI_FAIL;\r
+\r
+  /* network byte order */\r
+  port = ((const struct sockinet *)(const void *)sa)->si_port;\r
+  addr = (const char *)(const void *)sa + afd->a_off;\r
+\r
+  if (serv == NULL || servlen == 0) {\r
+    /*\r
+     * do nothing in this case.\r
+     * in case you are wondering if "&&" is more correct than\r
+     * "||" here: rfc2553bis-03 says that serv == NULL OR\r
+     * servlen == 0 means that the caller does not want the result.\r
+     */\r
+  } else {\r
+    if (flags & NI_NUMERICSERV)\r
+      sp = NULL;\r
+    else {\r
+      struct servent_data svd;\r
+//      struct servent sv;\r
+\r
+      (void)memset(&svd, 0, sizeof(svd));\r
+      sp = getservbyport_r(port,\r
+        (flags & NI_DGRAM) ? "udp" : "tcp", &sv, &svd);\r
+      endservent_r(&svd);\r
+    }\r
+    if (sp) {\r
+      if (strlen(sp->s_name) + 1 > servlen)\r
+        return EAI_MEMORY;\r
+      strlcpy(serv, sp->s_name, servlen);\r
+    } else {\r
+      snprintf(numserv, sizeof(numserv), "%u", ntohs(port));\r
+      if (strlen(numserv) + 1 > servlen)\r
+        return EAI_MEMORY;\r
+      strlcpy(serv, numserv, servlen);\r
+    }\r
+  }\r
+\r
+  switch (sa->sa_family) {\r
+  case AF_INET:\r
+    v4a = (u_int32_t)\r
+        ntohl(((const struct sockaddr_in *)\r
+        (const void *)sa)->sin_addr.s_addr);\r
+    if (IN_MULTICAST(v4a) || IN_EXPERIMENTAL(v4a))\r
+      flags |= NI_NUMERICHOST;\r
+    v4a >>= IN_CLASSA_NSHIFT;\r
+    if (v4a == 0)\r
+      flags |= NI_NUMERICHOST;\r
+    break;\r
 #ifdef INET6\r
-       case AF_INET6:\r
-           {\r
-               const struct sockaddr_in6 *sin6;\r
-               sin6 = (const struct sockaddr_in6 *)(const void *)sa;\r
-               switch (sin6->sin6_addr.s6_addr[0]) {\r
-               case 0x00:\r
-                       if (IN6_IS_ADDR_V4MAPPED(&sin6->sin6_addr))\r
-                               ;\r
-                       else if (IN6_IS_ADDR_LOOPBACK(&sin6->sin6_addr))\r
-                               ;\r
-                       else\r
-                               flags |= NI_NUMERICHOST;\r
-                       break;\r
-               default:\r
-                       if (IN6_IS_ADDR_LINKLOCAL(&sin6->sin6_addr)) {\r
-                               flags |= NI_NUMERICHOST;\r
-                       }\r
-                       else if (IN6_IS_ADDR_MULTICAST(&sin6->sin6_addr))\r
-                               flags |= NI_NUMERICHOST;\r
-                       break;\r
-               }\r
-           }\r
-               break;\r
+  case AF_INET6:\r
+      {\r
+    const struct sockaddr_in6 *sin6;\r
+    sin6 = (const struct sockaddr_in6 *)(const void *)sa;\r
+    switch (sin6->sin6_addr.s6_addr[0]) {\r
+    case 0x00:\r
+      if (IN6_IS_ADDR_V4MAPPED(&sin6->sin6_addr))\r
+        ;\r
+      else if (IN6_IS_ADDR_LOOPBACK(&sin6->sin6_addr))\r
+        ;\r
+      else\r
+        flags |= NI_NUMERICHOST;\r
+      break;\r
+    default:\r
+      if (IN6_IS_ADDR_LINKLOCAL(&sin6->sin6_addr)) {\r
+        flags |= NI_NUMERICHOST;\r
+      }\r
+      else if (IN6_IS_ADDR_MULTICAST(&sin6->sin6_addr))\r
+        flags |= NI_NUMERICHOST;\r
+      break;\r
+    }\r
+      }\r
+    break;\r
 #endif\r
-       }\r
-       if (host == NULL || hostlen == 0) {\r
-               /*\r
-                * do nothing in this case.\r
-                * in case you are wondering if "&&" is more correct than\r
-                * "||" here: rfc2553bis-03 says that host == NULL or\r
-                * hostlen == 0 means that the caller does not want the result.\r
-                */\r
-       } else if (flags & NI_NUMERICHOST) {\r
-               size_t numaddrlen;\r
-\r
-               /* NUMERICHOST and NAMEREQD conflicts with each other */\r
-               if (flags & NI_NAMEREQD)\r
-                       return EAI_NONAME;\r
-\r
-               switch(afd->a_af) {\r
+  }\r
+  if (host == NULL || hostlen == 0) {\r
+    /*\r
+     * do nothing in this case.\r
+     * in case you are wondering if "&&" is more correct than\r
+     * "||" here: rfc2553bis-03 says that host == NULL or\r
+     * hostlen == 0 means that the caller does not want the result.\r
+     */\r
+  } else if (flags & NI_NUMERICHOST) {\r
+    size_t numaddrlen;\r
+\r
+    /* NUMERICHOST and NAMEREQD conflicts with each other */\r
+    if (flags & NI_NAMEREQD)\r
+      return EAI_NONAME;\r
+\r
+    switch(afd->a_af) {\r
 #ifdef INET6\r
-               case AF_INET6:\r
-               {\r
-                       int error;\r
-\r
-                       if ((error = ip6_parsenumeric(sa, addr, host,\r
-                                                     hostlen, flags)) != 0)\r
-                               return(error);\r
-                       break;\r
-               }\r
+    case AF_INET6:\r
+    {\r
+      int error;\r
+\r
+      if ((error = ip6_parsenumeric(sa, addr, host,\r
+                  hostlen, flags)) != 0)\r
+        return(error);\r
+      break;\r
+    }\r
 #endif\r
-               default:\r
-                       if (inet_ntop(afd->a_af, addr, numaddr, sizeof(numaddr))\r
-                           == NULL)\r
-                               return EAI_SYSTEM;\r
-                       numaddrlen = strlen(numaddr);\r
-                       if (numaddrlen + 1 > hostlen) /* don't forget terminator */\r
-                               return EAI_MEMORY;\r
-                       strlcpy(host, numaddr, hostlen);\r
-                       break;\r
-               }\r
-       } else {\r
-               hp = gethostbyaddr(addr, afd->a_addrlen, afd->a_af);\r
-\r
-               if (hp) {\r
+    default:\r
+      if (inet_ntop(afd->a_af, addr, numaddr, sizeof(numaddr))\r
+          == NULL)\r
+        return EAI_SYSTEM;\r
+      numaddrlen = strlen(numaddr);\r
+      if (numaddrlen + 1 > hostlen) /* don't forget terminator */\r
+        return EAI_MEMORY;\r
+      strlcpy(host, numaddr, hostlen);\r
+      break;\r
+    }\r
+  } else {\r
+    hp = gethostbyaddr(addr, afd->a_addrlen, afd->a_af);\r
+\r
+    if (hp) {\r
 #if 0\r
-                       /*\r
-                        * commented out, since "for local host" is not\r
-                        * implemented here - see RFC2553 p30\r
-                        */\r
-                       if (flags & NI_NOFQDN) {\r
-                               char *p;\r
-                               p = strchr(hp->h_name, '.');\r
-                               if (p)\r
-                                       *p = '\0';\r
-                       }\r
+      /*\r
+       * commented out, since "for local host" is not\r
+       * implemented here - see RFC2553 p30\r
+       */\r
+      if (flags & NI_NOFQDN) {\r
+        char *p;\r
+        p = strchr(hp->h_name, '.');\r
+        if (p)\r
+          *p = '\0';\r
+      }\r
 #endif\r
-                       if (strlen(hp->h_name) + 1 > hostlen) {\r
-                               return EAI_MEMORY;\r
-                       }\r
-                       strlcpy(host, hp->h_name, hostlen);\r
-               } else {\r
-                       if (flags & NI_NAMEREQD)\r
-                               return EAI_NONAME;\r
-                       switch(afd->a_af) {\r
+      if (strlen(hp->h_name) + 1 > hostlen) {\r
+        return EAI_MEMORY;\r
+      }\r
+      strlcpy(host, hp->h_name, hostlen);\r
+    } else {\r
+      if (flags & NI_NAMEREQD)\r
+        return EAI_NONAME;\r
+      switch(afd->a_af) {\r
 #ifdef INET6\r
-                       case AF_INET6:\r
-                       {\r
-                               int error;\r
-\r
-                               if ((error = ip6_parsenumeric(sa, addr, host,\r
-                                                             hostlen,\r
-                                                             flags)) != 0)\r
-                                       return(error);\r
-                               break;\r
-                       }\r
+      case AF_INET6:\r
+      {\r
+        int error;\r
+\r
+        if ((error = ip6_parsenumeric(sa, addr, host,\r
+                    hostlen,\r
+                    flags)) != 0)\r
+          return(error);\r
+        break;\r
+      }\r
 #endif\r
-                       default:\r
-                               if (inet_ntop(afd->a_af, addr, host,\r
-                                   hostlen) == NULL)\r
-                                       return EAI_SYSTEM;\r
-                               break;\r
-                       }\r
-               }\r
-       }\r
-       return(0);\r
+      default:\r
+        if (inet_ntop(afd->a_af, addr, host,\r
+            hostlen) == NULL)\r
+          return EAI_SYSTEM;\r
+        break;\r
+      }\r
+    }\r
+  }\r
+  return(0);\r
 }\r
 \r
 #ifdef INET6\r
@@ -366,40 +376,40 @@ ip6_parsenumeric(
   int flags\r
   )\r
 {\r
-       size_t numaddrlen;\r
-       char numaddr[512];\r
-\r
-       _DIAGASSERT(sa != NULL);\r
-       _DIAGASSERT(addr != NULL);\r
-       _DIAGASSERT(host != NULL);\r
-\r
-       if (inet_ntop(AF_INET6, addr, numaddr, sizeof(numaddr)) == NULL)\r
-               return EAI_SYSTEM;\r
-\r
-       numaddrlen = strlen(numaddr);\r
-       if (numaddrlen + 1 > hostlen) /* don't forget terminator */\r
-               return EAI_OVERFLOW;\r
-       strlcpy(host, numaddr, hostlen);\r
-\r
-       if (((const struct sockaddr_in6 *)(const void *)sa)->sin6_scope_id) {\r
-               char zonebuf[MAXHOSTNAMELEN];\r
-               int zonelen;\r
-\r
-               zonelen = ip6_sa2str(\r
-                   (const struct sockaddr_in6 *)(const void *)sa,\r
-                   zonebuf, sizeof(zonebuf), flags);\r
-               if (zonelen < 0)\r
-                       return EAI_OVERFLOW;\r
-               if ((size_t) zonelen + 1 + numaddrlen + 1 > hostlen)\r
-                       return EAI_OVERFLOW;\r
-               /* construct <numeric-addr><delim><zoneid> */\r
-               memcpy(host + numaddrlen + 1, zonebuf,\r
-                   (size_t)zonelen);\r
-               host[numaddrlen] = SCOPE_DELIMITER;\r
-               host[numaddrlen + 1 + zonelen] = '\0';\r
-       }\r
-\r
-       return 0;\r
+  size_t numaddrlen;\r
+  char numaddr[512];\r
+\r
+  _DIAGASSERT(sa != NULL);\r
+  _DIAGASSERT(addr != NULL);\r
+  _DIAGASSERT(host != NULL);\r
+\r
+  if (inet_ntop(AF_INET6, addr, numaddr, sizeof(numaddr)) == NULL)\r
+    return EAI_SYSTEM;\r
+\r
+  numaddrlen = strlen(numaddr);\r
+  if (numaddrlen + 1 > hostlen) /* don't forget terminator */\r
+    return EAI_OVERFLOW;\r
+  strlcpy(host, numaddr, hostlen);\r
+\r
+  if (((const struct sockaddr_in6 *)(const void *)sa)->sin6_scope_id) {\r
+    char zonebuf[MAXHOSTNAMELEN];\r
+    int zonelen;\r
+\r
+    zonelen = ip6_sa2str(\r
+        (const struct sockaddr_in6 *)(const void *)sa,\r
+        zonebuf, sizeof(zonebuf), flags);\r
+    if (zonelen < 0)\r
+      return EAI_OVERFLOW;\r
+    if ((size_t) zonelen + 1 + numaddrlen + 1 > hostlen)\r
+      return EAI_OVERFLOW;\r
+    /* construct <numeric-addr><delim><zoneid> */\r
+    memcpy(host + numaddrlen + 1, zonebuf,\r
+        (size_t)zonelen);\r
+    host[numaddrlen] = SCOPE_DELIMITER;\r
+    host[numaddrlen + 1 + zonelen] = '\0';\r
+  }\r
+\r
+  return 0;\r
 }\r
 \r
 /* ARGSUSED */\r
@@ -411,43 +421,47 @@ ip6_sa2str(
   int flags\r
   )\r
 {\r
-       unsigned int ifindex;\r
-       const struct in6_addr *a6;\r
-       int n;\r
+#if 0\r
+  unsigned int ifindex;\r
+  const struct in6_addr *a6;\r
+#endif\r
+  int n;\r
 \r
-       _DIAGASSERT(sa6 != NULL);\r
-       _DIAGASSERT(buf != NULL);\r
+  _DIAGASSERT(sa6 != NULL);\r
+  _DIAGASSERT(buf != NULL);\r
 \r
-       ifindex = (unsigned int)sa6->sin6_scope_id;\r
-       a6 = &sa6->sin6_addr;\r
+#if 0\r
+  ifindex = (unsigned int)sa6->sin6_scope_id;\r
+  a6 = &sa6->sin6_addr;\r
+#endif\r
 \r
 #ifdef NI_NUMERICSCOPE\r
-       if ((flags & NI_NUMERICSCOPE) != 0) {\r
-               n = snprintf(buf, bufsiz, "%u", sa6->sin6_scope_id);\r
-               if ((n < 0) || ((size_t)n >= bufsiz))\r
-                       return -1;\r
-               else\r
-                       return n;\r
-       }\r
+  if ((flags & NI_NUMERICSCOPE) != 0) {\r
+    n = snprintf(buf, bufsiz, "%u", sa6->sin6_scope_id);\r
+    if ((n < 0) || ((size_t)n >= bufsiz))\r
+      return -1;\r
+    else\r
+      return n;\r
+  }\r
 #endif\r
 \r
 #if 0\r
-       /* if_indextoname() does not take buffer size.  not a good api... */\r
-       if ((IN6_IS_ADDR_LINKLOCAL(a6) || IN6_IS_ADDR_MC_LINKLOCAL(a6)) &&\r
-           bufsiz >= IF_NAMESIZE) {\r
-               char *p = if_indextoname(ifindex, buf);\r
-               if (p) {\r
-                       return(strlen(p));\r
-               }\r
-       }\r
+  /* if_indextoname() does not take buffer size.  not a good api... */\r
+  if ((IN6_IS_ADDR_LINKLOCAL(a6) || IN6_IS_ADDR_MC_LINKLOCAL(a6)) &&\r
+      bufsiz >= IF_NAMESIZE) {\r
+    char *p = if_indextoname(ifindex, buf);\r
+    if (p) {\r
+      return(strlen(p));\r
+    }\r
+  }\r
 #endif  //  0\r
 \r
-       /* last resort */\r
-       n = snprintf(buf, bufsiz, "%u", sa6->sin6_scope_id);\r
-       if (n < 0 || (size_t) n >= bufsiz)\r
-               return -1;\r
-       else\r
-               return n;\r
+  /* last resort */\r
+  n = snprintf(buf, bufsiz, "%u", sa6->sin6_scope_id);\r
+  if (n < 0 || (size_t) n >= bufsiz)\r
+    return -1;\r
+  else\r
+    return n;\r
 }\r
 #endif /* INET6 */\r
 \r
@@ -470,73 +484,73 @@ getnameinfo_link (
   int flags\r
   )\r
 {\r
-       const struct sockaddr_dl *sdl =\r
-           (const struct sockaddr_dl *)(const void *)sa;\r
-//     const struct ieee1394_hwaddr *iha;\r
-       int n;\r
-\r
-       if (serv != NULL && servlen > 0)\r
-               *serv = '\0';\r
-\r
-       if (sdl->sdl_nlen == 0 && sdl->sdl_alen == 0 && sdl->sdl_slen == 0) {\r
-               n = snprintf(host, hostlen, "link#%u", sdl->sdl_index);\r
-               if (n < 0 || (socklen_t) n > hostlen) {\r
-                       *host = '\0';\r
-                       return EAI_MEMORY;\r
-               }\r
-               return 0;\r
-       }\r
+  const struct sockaddr_dl *sdl =\r
+      (const struct sockaddr_dl *)(const void *)sa;\r
+//  const struct ieee1394_hwaddr *iha;\r
+  int n;\r
+\r
+  if (serv != NULL && servlen > 0)\r
+    *serv = '\0';\r
+\r
+  if (sdl->sdl_nlen == 0 && sdl->sdl_alen == 0 && sdl->sdl_slen == 0) {\r
+    n = snprintf(host, hostlen, "link#%u", sdl->sdl_index);\r
+    if (n < 0 || (socklen_t) n > hostlen) {\r
+      *host = '\0';\r
+      return EAI_MEMORY;\r
+    }\r
+    return 0;\r
+  }\r
 \r
 #if 0\r
-       switch (sdl->sdl_type) {\r
+  switch (sdl->sdl_type) {\r
 #ifdef IFT_ECONET\r
-       case IFT_ECONET:\r
-               if (sdl->sdl_alen < 2)\r
-                       return EAI_FAMILY;\r
-               if (CLLADDR(sdl)[1] == 0)\r
-                       n = snprintf(host, hostlen, "%u", CLLADDR(sdl)[0]);\r
-               else\r
-                       n = snprintf(host, hostlen, "%u.%u",\r
-                           CLLADDR(sdl)[1], CLLADDR(sdl)[0]);\r
-               if (n < 0 || (socklen_t) n >= hostlen) {\r
-                       *host = '\0';\r
-                       return EAI_MEMORY;\r
-               } else\r
-                       return 0;\r
+  case IFT_ECONET:\r
+    if (sdl->sdl_alen < 2)\r
+      return EAI_FAMILY;\r
+    if (CLLADDR(sdl)[1] == 0)\r
+      n = snprintf(host, hostlen, "%u", CLLADDR(sdl)[0]);\r
+    else\r
+      n = snprintf(host, hostlen, "%u.%u",\r
+          CLLADDR(sdl)[1], CLLADDR(sdl)[0]);\r
+    if (n < 0 || (socklen_t) n >= hostlen) {\r
+      *host = '\0';\r
+      return EAI_MEMORY;\r
+    } else\r
+      return 0;\r
 #endif\r
-       case IFT_IEEE1394:\r
-               if (sdl->sdl_alen < sizeof(iha->iha_uid))\r
-                       return EAI_FAMILY;\r
-               iha =\r
-                   (const struct ieee1394_hwaddr *)(const void *)CLLADDR(sdl);\r
-               return hexname(iha->iha_uid, sizeof(iha->iha_uid),\r
-                   host, hostlen);\r
-       /*\r
-        * The following have zero-length addresses.\r
-        * IFT_ATM      (net/if_atmsubr.c)\r
-        * IFT_FAITH    (net/if_faith.c)\r
-        * IFT_GIF      (net/if_gif.c)\r
-        * IFT_LOOP     (net/if_loop.c)\r
-        * IFT_PPP      (net/if_ppp.c, net/if_spppsubr.c)\r
-        * IFT_SLIP     (net/if_sl.c, net/if_strip.c)\r
-        * IFT_STF      (net/if_stf.c)\r
-        * IFT_L2VLAN   (net/if_vlan.c)\r
-        * IFT_PROPVIRTUAL (net/if_bridge.h>\r
-        */\r
-       /*\r
-        * The following use IPv4 addresses as link-layer addresses:\r
-        * IFT_OTHER    (net/if_gre.c)\r
-        */\r
-       case IFT_ARCNET: /* default below is believed correct for all these. */\r
-       case IFT_ETHER:\r
-       case IFT_FDDI:\r
-       case IFT_HIPPI:\r
-       case IFT_ISO88025:\r
-       default:\r
+  case IFT_IEEE1394:\r
+    if (sdl->sdl_alen < sizeof(iha->iha_uid))\r
+      return EAI_FAMILY;\r
+    iha =\r
+        (const struct ieee1394_hwaddr *)(const void *)CLLADDR(sdl);\r
+    return hexname(iha->iha_uid, sizeof(iha->iha_uid),\r
+        host, hostlen);\r
+  /*\r
+   * The following have zero-length addresses.\r
+   * IFT_ATM  (net/if_atmsubr.c)\r
+   * IFT_FAITH  (net/if_faith.c)\r
+   * IFT_GIF  (net/if_gif.c)\r
+   * IFT_LOOP (net/if_loop.c)\r
+   * IFT_PPP  (net/if_ppp.c, net/if_spppsubr.c)\r
+   * IFT_SLIP (net/if_sl.c, net/if_strip.c)\r
+   * IFT_STF  (net/if_stf.c)\r
+   * IFT_L2VLAN (net/if_vlan.c)\r
+   * IFT_PROPVIRTUAL (net/if_bridge.h>\r
+   */\r
+  /*\r
+   * The following use IPv4 addresses as link-layer addresses:\r
+   * IFT_OTHER  (net/if_gre.c)\r
+   */\r
+  case IFT_ARCNET: /* default below is believed correct for all these. */\r
+  case IFT_ETHER:\r
+  case IFT_FDDI:\r
+  case IFT_HIPPI:\r
+  case IFT_ISO88025:\r
+  default:\r
 #endif  //  0\r
-               return hexname((const u_int8_t *)CLLADDR(sdl),\r
-                   (size_t)sdl->sdl_alen, host, hostlen);\r
-//     }\r
+    return hexname((const u_int8_t *)CLLADDR(sdl),\r
+        (size_t)sdl->sdl_alen, host, hostlen);\r
+//  }\r
 }\r
 \r
 static\r
@@ -548,20 +562,20 @@ hexname(
   socklen_t hostlen\r
   )\r
 {\r
-       int n;\r
-       size_t i;\r
-       char *outp = host;\r
-\r
-       *outp = '\0';\r
-       for (i = 0; i < len; i++) {\r
-               n = snprintf(outp, hostlen, "%s%02x",\r
-                   i ? ":" : "", cp[i]);\r
-               if (n < 0 || (socklen_t) n >= hostlen) {\r
-                       *host = '\0';\r
-                       return EAI_MEMORY;\r
-               }\r
-               outp += n;\r
-               hostlen -= n;\r
-       }\r
-       return 0;\r
+  int n;\r
+  size_t i;\r
+  char *outp = host;\r
+\r
+  *outp = '\0';\r
+  for (i = 0; i < len; i++) {\r
+    n = snprintf(outp, hostlen, "%s%02x",\r
+        i ? ":" : "", cp[i]);\r
+    if (n < 0 || (socklen_t) n >= hostlen) {\r
+      *host = '\0';\r
+      return EAI_MEMORY;\r
+    }\r
+    outp += n;\r
+    hostlen -= n;\r
+  }\r
+  return 0;\r
 }\r
index cddda3e150f8b0be0f190c6e67be8694bc752329..7c327fac2e40985b0ab40cd3673f2a80fda34714 100644 (file)
@@ -1,3 +1,13 @@
+/** @file\r
+  Copyright (c) 1999 - 2014, Intel Corporation. All rights reserved.<BR>\r
+  This program and the accompanying materials are licensed and made available\r
+  under the terms and conditions of the BSD License which accompanies this\r
+  distribution.  The full text of the license may be found at\r
+  http://opensource.org/licenses/bsd-license.php.\r
+\r
+  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
+  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
+**/\r
 /*\r
  * Copyright (c) 1985, 1993\r
  *    The Regents of the University of California.  All rights reserved.\r
@@ -168,7 +178,7 @@ res_hnok(
     const char *dn\r
     )\r
 {\r
-    int ppch = '\0', pch = PERIOD, ch = *dn++;\r
+    int pch = PERIOD, ch = *dn++;\r
 \r
     while (ch != '\0') {\r
         int nch = *dn++;\r
@@ -185,7 +195,8 @@ res_hnok(
             if (!middlechar(ch))\r
                 return (0);\r
         }\r
-        ppch = pch, pch = ch, ch = nch;\r
+        pch = ch;\r
+        ch = nch;\r
     }\r
     return (1);\r
 }\r
index 3201f312e6834c3ad73b0c78c801f2e96c2502e8..d81d7d6f1518dfe1a9870fa8abff6e887110a063 100644 (file)
@@ -1,3 +1,13 @@
+/** @file\r
+  Copyright (c) 1999 - 2014, Intel Corporation. All rights reserved.<BR>\r
+  This program and the accompanying materials are licensed and made available\r
+  under the terms and conditions of the BSD License which accompanies this\r
+  distribution.  The full text of the license may be found at\r
+  http://opensource.org/licenses/bsd-license.php.\r
+\r
+  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
+  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
+**/\r
 /*\r
  * Copyright (c) 1996 by Internet Software Consortium.\r
  *\r
@@ -100,7 +110,7 @@ int
 res_mkupdate(ns_updrec *rrecp_in, u_char *buf, int buflen) {\r
     ns_updrec *rrecp_start = rrecp_in;\r
     HEADER *hp;\r
-    u_char *cp, *sp1, *sp2, *startp, *endp;\r
+    u_char *cp, *sp2, *startp, *endp;\r
     int n, i, soanum, multiline;\r
     ns_updrec *rrecp;\r
     struct in_addr ina;\r
@@ -125,7 +135,6 @@ res_mkupdate(ns_updrec *rrecp_in, u_char *buf, int buflen) {
     hp->id = htons(++_res.id);\r
     hp->opcode = ns_o_update;\r
     hp->rcode = NOERROR;\r
-    sp1 = buf + 2*INT16SZ;  /* save pointer to zocount */\r
     cp = buf + HFIXEDSZ;\r
     buflen -= HFIXEDSZ;\r
     dpp = dnptrs;\r
index ed71194163df0c1d7d93a48c6616f4f720d3c868..84646e726a49df04a4801bf9517646015a9d1c2f 100644 (file)
@@ -1,11 +1,11 @@
 /** @file\r
   Implement the IP4 driver support for the socket layer.\r
 \r
-  Copyright (c) 2011, Intel Corporation\r
-  All rights reserved. This program and the accompanying materials\r
-  are licensed and made available under the terms and conditions of the BSD License\r
-  which accompanies this distribution.  The full text of the license may be found at\r
-  http://opensource.org/licenses/bsd-license.php\r
+  Copyright (c) 2011 - 2014, Intel Corporation. All rights reserved.<BR>\r
+  This program and the accompanying materials are licensed and made available\r
+  under the terms and conditions of the BSD License which accompanies this\r
+  distribution.  The full text of the license may be found at\r
+  http://opensource.org/licenses/bsd-license.php.\r
 \r
   THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
   WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
@@ -430,13 +430,13 @@ EslIp4PortAllocate (
   @param [in] pPort           Address of an ::ESL_PORT structure.\r
 \r
   @param [in] pPacket         Address of an ::ESL_PACKET structure.\r
-  \r
+\r
   @param [in] pbConsumePacket Address of a BOOLEAN indicating if the packet is to be consumed\r
-  \r
+\r
   @param [in] BufferLength    Length of the the buffer\r
-  \r
+\r
   @param [in] pBuffer         Address of a buffer to receive the data.\r
-  \r
+\r
   @param [in] pDataLength     Number of received data bytes in the buffer.\r
 \r
   @param [out] pAddress       Network address to receive the remote system address\r
@@ -653,17 +653,15 @@ EslIp4RxComplete (
   )\r
 {\r
   size_t LengthInBytes;\r
-  ESL_PORT * pPort;\r
   ESL_PACKET * pPacket;\r
   EFI_IP4_RECEIVE_DATA * pRxData;\r
   EFI_STATUS Status;\r
-  \r
+\r
   DBG_ENTER ( );\r
-  \r
+\r
   //\r
   //  Get the operation status.\r
   //\r
-  pPort = pIo->pPort;\r
   Status = pIo->Token.Ip4Rx.Status;\r
 \r
   //\r
@@ -672,7 +670,7 @@ EslIp4RxComplete (
   pRxData = pIo->Token.Ip4Rx.Packet.RxData;\r
   LengthInBytes = pRxData->HeaderLength + pRxData->DataLength;\r
 \r
-  //\r
+  //{{\r
   //      +--------------------+   +----------------------+\r
   //      | ESL_IO_MGMT        |   |      Data Buffer     |\r
   //      |                    |   |     (Driver owned)   |\r
@@ -692,7 +690,7 @@ EslIp4RxComplete (
   //\r
   //\r
   //  Save the data in the packet\r
-  //\r
+  //}}\r
   pPacket = pIo->pPacket;\r
   pPacket->Op.Ip4Rx.pRxData = pRxData;\r
 \r
@@ -717,7 +715,7 @@ EslIp4RxComplete (
   that the socket is configured.\r
 \r
   @param [in] pSocket         Address of an ::ESL_SOCKET structure\r
-  \r
+\r
   @retval EFI_SUCCESS - The port is connected\r
   @retval EFI_NOT_STARTED - The port is not connected\r
 \r
@@ -904,7 +902,7 @@ EslIp4RxComplete (
   //  Determine the socket configuration status\r
   //\r
   Status = pSocket->bConfigured ? EFI_SUCCESS : EFI_NOT_STARTED;\r
-  \r
+\r
   //\r
   //  Return the port connected state.\r
   //\r
@@ -1188,9 +1186,9 @@ EslIp4TxComplete (
   ESL_PACKET * pPacket;\r
   ESL_SOCKET * pSocket;\r
   EFI_STATUS Status;\r
-  \r
+\r
   DBG_ENTER ( );\r
-  \r
+\r
   //\r
   //  Locate the active transmit packet\r
   //\r
index 7ece38d52137329e74d800f8830bd32ded86be03..943af946bce68fe70bdbeaff228be31ffdd5c99f 100644 (file)
@@ -1,18 +1,18 @@
 /** @file\r
   Implement the TCP4 driver support for the socket layer.\r
 \r
-  Copyright (c) 2011, Intel Corporation\r
-  All rights reserved. This program and the accompanying materials\r
-  are licensed and made available under the terms and conditions of the BSD License\r
-  which accompanies this distribution.  The full text of the license may be found at\r
-  http://opensource.org/licenses/bsd-license.php\r
+  Copyright (c) 2011 - 2014, Intel Corporation. All rights reserved.<BR>\r
+  This program and the accompanying materials are licensed and made available\r
+  under the terms and conditions of the BSD License which accompanies this\r
+  distribution.  The full text of the license may be found at\r
+  http://opensource.org/licenses/bsd-license.php.\r
 \r
   THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
   WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
 \r
 \r
   \section ConnectionManagement Connection Management\r
-  \r
+\r
   The ::EslTcp4Listen routine initially places the SOCK_STREAM or\r
   SOCK_SEQPACKET socket into a listen state.   When a remote machine\r
   makes a connection to the socket, the TCPv4 network layer calls\r
@@ -487,7 +487,7 @@ EslTcp4ConnectStart (
   EFI_STATUS Status;\r
 \r
   DBG_ENTER ( );\r
-  \r
+\r
   //\r
   //  Determine if any more local adapters are available\r
   //\r
@@ -776,7 +776,7 @@ EslTcp4Listen (
       //\r
       pPort = pNextPort;\r
     }\r
-    \r
+\r
     //\r
     //  Determine if any ports are in the listen state\r
     //\r
@@ -840,7 +840,6 @@ EslTcp4ListenComplete (
   EFI_HANDLE ChildHandle;\r
   struct sockaddr_in LocalAddress;\r
   EFI_TCP4_CONFIG_DATA * pConfigData;\r
-  ESL_LAYER * pLayer;\r
   ESL_PORT * pNewPort;\r
   ESL_SOCKET * pNewSocket;\r
   ESL_SOCKET * pSocket;\r
@@ -869,7 +868,6 @@ EslTcp4ListenComplete (
     //  Allocate a socket for this connection\r
     //\r
     ChildHandle = NULL;\r
-    pLayer = &mEslLayer;\r
     Status = EslSocketAllocate ( &ChildHandle,\r
                                  DEBUG_CONNECTION,\r
                                  &pNewSocket );\r
@@ -1041,7 +1039,7 @@ EslTcp4ListenComplete (
     //  Process:\r
     //    Call close\r
     //    Release the resources\r
-    \r
+\r
   }\r
 \r
   DBG_EXIT ( );\r
@@ -1151,7 +1149,7 @@ EslTcp4LocalAddressSet (
     //  Determine if the default address is used\r
     //\r
     pAccessPoint->UseDefaultAddress = (BOOLEAN)( 0 == pIpAddress->sin_addr.s_addr );\r
-    \r
+\r
     //\r
     //  Set the subnet mask\r
     //\r
@@ -1345,7 +1343,7 @@ EslTcp4PortAllocate (
 \r
   This routine is called by ::EslSocketPortClose.\r
   See the \ref PortCloseStateMachine section.\r
-  \r
+\r
   @param [in] pPort       Address of an ::ESL_PORT structure.\r
 \r
   @retval EFI_SUCCESS     The port is closed\r
@@ -1360,7 +1358,7 @@ EslTcp4PortClose (
   UINTN DebugFlags;\r
   ESL_TCP4_CONTEXT * pTcp4;\r
   EFI_STATUS Status;\r
-  \r
+\r
   DBG_ENTER ( );\r
 \r
   //\r
@@ -1504,13 +1502,13 @@ EslTcp4PortCloseOp (
   @param [in] pPort           Address of an ::ESL_PORT structure.\r
 \r
   @param [in] pPacket         Address of an ::ESL_PACKET structure.\r
-  \r
+\r
   @param [in] pbConsumePacket Address of a BOOLEAN indicating if the packet is to be consumed\r
-  \r
+\r
   @param [in] BufferLength    Length of the the buffer\r
-  \r
+\r
   @param [in] pBuffer         Address of a buffer to receive the data.\r
-  \r
+\r
   @param [in] pDataLength     Number of received data bytes in the buffer.\r
 \r
   @param [out] pAddress       Network address to receive the remote system address\r
@@ -1888,13 +1886,13 @@ EslTcp4RxStart (
   during the current transmission attempt.\r
 \r
   @param [in] pSocket         Address of an ::ESL_SOCKET structure\r
-  \r
+\r
   @param [in] Flags           Message control flags\r
-  \r
+\r
   @param [in] BufferLength    Length of the the buffer\r
-  \r
+\r
   @param [in] pBuffer         Address of a buffer to receive the data.\r
-  \r
+\r
   @param [in] pDataLength     Number of received data bytes in the buffer.\r
 \r
   @param [in] pAddress        Network address of the remote system address\r
@@ -1924,7 +1922,6 @@ EslTcp4TxBuffer (
   ESL_PACKET ** ppQueueHead;\r
   ESL_PACKET ** ppQueueTail;\r
   ESL_PACKET * pPreviousPacket;\r
-  ESL_TCP4_CONTEXT * pTcp4;\r
   size_t * pTxBytes;\r
   EFI_TCP4_TRANSMIT_DATA * pTxData;\r
   EFI_STATUS Status;\r
@@ -1951,7 +1948,6 @@ EslTcp4TxBuffer (
       //\r
       //  Determine the queue head\r
       //\r
-      pTcp4 = &pPort->Context.Tcp4;\r
       bUrgent = (BOOLEAN)( 0 != ( Flags & MSG_OOB ));\r
       bUrgentQueue = bUrgent\r
                     && ( !pSocket->bOobInLine )\r
@@ -2142,7 +2138,7 @@ EslTcp4TxComplete (
   ESL_PORT * pPort;\r
   ESL_SOCKET * pSocket;\r
   EFI_STATUS Status;\r
-  \r
+\r
   DBG_ENTER ( );\r
 \r
   //\r
index 21c4109d6ea3fd5c7e1c4c6ac073d44e71978d78..0f6d2d6ac93c24f85fa91dc99979a20590ea13f5 100644 (file)
@@ -1,18 +1,18 @@
 /** @file\r
   Implement the TCP6 driver support for the socket layer.\r
 \r
-  Copyright (c) 2011, Intel Corporation\r
-  All rights reserved. This program and the accompanying materials\r
-  are licensed and made available under the terms and conditions of the BSD License\r
-  which accompanies this distribution.  The full text of the license may be found at\r
-  http://opensource.org/licenses/bsd-license.php\r
+  Copyright (c) 2011 - 2014, Intel Corporation. All rights reserved.<BR>\r
+  This program and the accompanying materials are licensed and made available\r
+  under the terms and conditions of the BSD License which accompanies this\r
+  distribution.  The full text of the license may be found at\r
+  http://opensource.org/licenses/bsd-license.php.\r
 \r
   THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
   WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
 \r
 \r
   \section ConnectionManagement Connection Management\r
-  \r
+\r
   The ::EslTcp6Listen routine initially places the SOCK_STREAM or\r
   SOCK_SEQPACKET socket into a listen state.   When a remote machine\r
   makes a connection to the socket, the TCPv6 network layer calls\r
@@ -398,55 +398,55 @@ EslTcp6ConnectPoll (
       case EFI_DEVICE_ERROR:\r
         pSocket->errno = EIO;\r
         break;\r
-      \r
+\r
       case EFI_ABORTED:\r
         pSocket->errno = ECONNABORTED;\r
         break;\r
-      \r
+\r
       case EFI_ACCESS_DENIED:\r
         pSocket->errno = EACCES;\r
         break;\r
-      \r
+\r
       case EFI_CONNECTION_RESET:\r
         pSocket->errno = ECONNRESET;\r
         break;\r
-      \r
+\r
       case EFI_INVALID_PARAMETER:\r
         pSocket->errno = EADDRNOTAVAIL;\r
         break;\r
-      \r
+\r
       case EFI_HOST_UNREACHABLE:\r
       case EFI_NO_RESPONSE:\r
         pSocket->errno = EHOSTUNREACH;\r
         break;\r
-      \r
+\r
       case EFI_NO_MAPPING:\r
         pSocket->errno = EAFNOSUPPORT;\r
         break;\r
-      \r
+\r
       case EFI_NO_MEDIA:\r
       case EFI_NETWORK_UNREACHABLE:\r
         pSocket->errno = ENETDOWN;\r
         break;\r
-      \r
+\r
       case EFI_OUT_OF_RESOURCES:\r
         pSocket->errno = ENOBUFS;\r
         break;\r
-      \r
+\r
       case EFI_PORT_UNREACHABLE:\r
       case EFI_PROTOCOL_UNREACHABLE:\r
       case EFI_CONNECTION_REFUSED:\r
         pSocket->errno = ECONNREFUSED;\r
         break;\r
-      \r
+\r
       case EFI_SUCCESS:\r
         pSocket->errno = 0;\r
         break;\r
-      \r
+\r
       case EFI_TIMEOUT:\r
         pSocket->errno = ETIMEDOUT;\r
         break;\r
-      \r
+\r
       case EFI_UNSUPPORTED:\r
         pSocket->errno = EOPNOTSUPP;\r
         break;\r
@@ -505,7 +505,7 @@ EslTcp6ConnectStart (
   EFI_STATUS Status;\r
 \r
   DBG_ENTER ( );\r
-  \r
+\r
   //\r
   //  Determine if any more local adapters are available\r
   //\r
@@ -601,7 +601,7 @@ EslTcp6ConnectStart (
       //  Status to errno translation gets done in EslTcp4ConnectPoll\r
       //\r
       pTcp6->ConnectToken.CompletionToken.Status = Status;\r
-      \r
+\r
       //\r
       //  Continue with the next port\r
       //\r
@@ -807,7 +807,7 @@ EslTcp6Listen (
       //\r
       pPort = pNextPort;\r
     }\r
-    \r
+\r
     //\r
     //  Determine if any ports are in the listen state\r
     //\r
@@ -871,7 +871,6 @@ EslTcp6ListenComplete (
   EFI_HANDLE ChildHandle;\r
   struct sockaddr_in6 LocalAddress;\r
   EFI_TCP6_CONFIG_DATA * pConfigData;\r
-  ESL_LAYER * pLayer;\r
   ESL_PORT * pNewPort;\r
   ESL_SOCKET * pNewSocket;\r
   ESL_SOCKET * pSocket;\r
@@ -900,7 +899,6 @@ EslTcp6ListenComplete (
     //  Allocate a socket for this connection\r
     //\r
     ChildHandle = NULL;\r
-    pLayer = &mEslLayer;\r
     Status = EslSocketAllocate ( &ChildHandle,\r
                                  DEBUG_CONNECTION,\r
                                  &pNewSocket );\r
@@ -1098,7 +1096,7 @@ EslTcp6ListenComplete (
     //  Process:\r
     //    Call close\r
     //    Release the resources\r
-    \r
+\r
   }\r
 \r
   DBG_EXIT ( );\r
@@ -1397,7 +1395,7 @@ EslTcp6PortAllocate (
 \r
   This routine is called by ::EslSocketPortClose.\r
   See the \ref PortCloseStateMachine section.\r
-  \r
+\r
   @param [in] pPort       Address of an ::ESL_PORT structure.\r
 \r
   @retval EFI_SUCCESS     The port is closed\r
@@ -1412,7 +1410,7 @@ EslTcp6PortClose (
   UINTN DebugFlags;\r
   ESL_TCP6_CONTEXT * pTcp6;\r
   EFI_STATUS Status;\r
-  \r
+\r
   DBG_ENTER ( );\r
 \r
   //\r
@@ -1556,13 +1554,13 @@ EslTcp6PortCloseOp (
   @param [in] pPort           Address of an ::ESL_PORT structure.\r
 \r
   @param [in] pPacket         Address of an ::ESL_PACKET structure.\r
-  \r
+\r
   @param [in] pbConsumePacket Address of a BOOLEAN indicating if the packet is to be consumed\r
-  \r
+\r
   @param [in] BufferLength    Length of the the buffer\r
-  \r
+\r
   @param [in] pBuffer         Address of a buffer to receive the data.\r
-  \r
+\r
   @param [in] pDataLength     Number of received data bytes in the buffer.\r
 \r
   @param [out] pAddress       Network address to receive the remote system address\r
@@ -1957,13 +1955,13 @@ EslTcp6RxStart (
   during the current transmission attempt.\r
 \r
   @param [in] pSocket         Address of an ::ESL_SOCKET structure\r
-  \r
+\r
   @param [in] Flags           Message control flags\r
-  \r
+\r
   @param [in] BufferLength    Length of the the buffer\r
-  \r
+\r
   @param [in] pBuffer         Address of a buffer to receive the data.\r
-  \r
+\r
   @param [in] pDataLength     Number of received data bytes in the buffer.\r
 \r
   @param [in] pAddress        Network address of the remote system address\r
@@ -1993,7 +1991,6 @@ EslTcp6TxBuffer (
   ESL_PACKET ** ppQueueHead;\r
   ESL_PACKET ** ppQueueTail;\r
   ESL_PACKET * pPreviousPacket;\r
-  ESL_TCP6_CONTEXT * pTcp6;\r
   size_t * pTxBytes;\r
   EFI_TCP6_TRANSMIT_DATA * pTxData;\r
   EFI_STATUS Status;\r
@@ -2020,7 +2017,6 @@ EslTcp6TxBuffer (
       //\r
       //  Determine the queue head\r
       //\r
-      pTcp6 = &pPort->Context.Tcp6;\r
       bUrgent = (BOOLEAN)( 0 != ( Flags & MSG_OOB ));\r
       bUrgentQueue = bUrgent\r
                     && ( !pSocket->bOobInLine )\r
@@ -2211,7 +2207,7 @@ EslTcp6TxComplete (
   ESL_PORT * pPort;\r
   ESL_SOCKET * pSocket;\r
   EFI_STATUS Status;\r
-  \r
+\r
   DBG_ENTER ( );\r
 \r
   //\r
index 73254d2e56248e7d3e95dcfb3cc3b43afe29462c..60def644e91691a06610a940ccea1a2b1231f8ef 100644 (file)
@@ -292,17 +292,17 @@ FIFO_Dequeue (
     SizeOfElement = Self->ElementSize;                                // Get size of this FIFO's elements\r
     Count         = MIN(Count, Self->Count(Self, AsElements));        // Lesser of requested or actual\r
 \r
-    QPtr = (UINTN)Self->Queue + (RDex * Self->ElementSize);           // Point to Read location in FIFO\r
+    QPtr = (UINTN)Self->Queue + (RDex * SizeOfElement);           // Point to Read location in FIFO\r
     for(i = 0; i < Count; ++i) {                                      // Iterate Count times...\r
-      (void)CopyMem(pElement, (const void *)QPtr, Self->ElementSize);   // Copy element from FIFO to caller's buffer\r
+      (void)CopyMem(pElement, (const void *)QPtr, SizeOfElement);   // Copy element from FIFO to caller's buffer\r
       RDex = (UINT32)ModuloIncrement(RDex, Self->NumElements);          // Increment Read Index\r
       if(RDex == 0) {                                                   // If the index wrapped\r
         QPtr = (UINTN)Self->Queue;                                        // Point back to beginning of data\r
       }\r
       else {                                                            // Otherwise\r
-        QPtr += Self->ElementSize;                                        // Point to the next element in FIFO\r
+        QPtr += SizeOfElement;                                        // Point to the next element in FIFO\r
       }\r
-      pElement = (char*)pElement + Self->ElementSize;                   // Point to next element in caller's buffer\r
+      pElement = (char*)pElement + SizeOfElement;                   // Point to next element in caller's buffer\r
     }                                                                 // Iterate: for loop\r
     if(Consume) {                                                     // If caller requests data consumption\r
       Self->ReadIndex = RDex;                                           // Set FIFO's Read Index to new Index\r
index 0e80ff0eaf7538547ae7269aa6e077e92f273221..2e61cd1e80afda3c93430f62777b47673c06b89a 100644 (file)
@@ -75,7 +75,7 @@ IIO_GetInChar (
 {\r
   cIIO             *This;\r
   cFIFO            *InBuf;\r
-  EFI_STATUS        Status;\r
+  size_t            Status;\r
   ssize_t           NumRead;\r
   wint_t            RetVal;\r
   wchar_t           InChar;\r
@@ -92,8 +92,10 @@ IIO_GetInChar (
   }\r
   if(BufCnt > 0) {\r
     Status = InBuf->Read(InBuf, &InChar, 1);\r
-    --BufCnt;\r
-    NumRead = 1;\r
+    if (Status > 0) {\r
+      --BufCnt;\r
+      NumRead = 1;\r
+    }\r
   }\r
   else {\r
     NumRead = filp->f_ops->fo_read(filp, &filp->f_offset, sizeof(wchar_t), &InChar);\r
index 927f4f4ff3f152ad32ba3079267c8c042bd12e6c..ef52cc890e24c5203be5d707bc73825143eb2e5b 100644 (file)
@@ -8,7 +8,7 @@
   It is the responsibility of the caller, or higher level function, to perform\r
   any necessary translation between wide and narrow characters.\r
 \r
-  Copyright (c) 2012, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2012 - 2014, Intel Corporation. All rights reserved.<BR>\r
   This program and the accompanying materials are licensed and made available\r
   under the terms and conditions of the BSD License which accompanies this\r
   distribution.  The full text of the license may be found at\r
@@ -63,7 +63,6 @@ IIO_WriteOne(struct __filedes *filp, cFIFO *OBuf, wchar_t InCh)
   UINT32              CurRow;       // Current cursor row on the screen\r
   UINT32              PrevColumn;   // Previous column.  Used to detect wrapping.\r
   UINT32              AdjColumn;    // Current cursor column on the screen\r
-  UINT32              AdjRow;       // Current cursor row on the screen\r
 \r
   RetVal    = -1;\r
   wcb       = wc;\r
@@ -79,7 +78,6 @@ IIO_WriteOne(struct __filedes *filp, cFIFO *OBuf, wchar_t InCh)
     CurRow    = This->CurrentXY.Row;\r
 \r
     numW      = 1;          // The majority of characters buffer one character\r
-    AdjRow    = 0;          // Most characters just cause horizontal movement\r
     AdjColumn = 0;\r
     if(OFlag & OPOST) {\r
       /* Perform output processing */\r
@@ -127,7 +125,6 @@ IIO_WriteOne(struct __filedes *filp, cFIFO *OBuf, wchar_t InCh)
             numW  = 2;\r
             CurColumn = 0;\r
           }\r
-          AdjRow = 1;\r
           break;                      //}}\r
 \r
         case CHAR_BACKSPACE:          //{{\r
index bcae9c82399e657a364d002b8374678a7ce7474c..aab81cdaa8ef500b9b30a6879aeb585521550c02 100644 (file)
@@ -37,7 +37,6 @@ IIO_NonCanonRead (
   cIIO           *This;\r
   cFIFO          *InBuf;\r
   struct termios *Termio;\r
-  EFI_STATUS      Status;\r
   ssize_t         NumRead;\r
   cc_t            tioMin;\r
   cc_t            tioTime;\r
@@ -74,7 +73,7 @@ IIO_NonCanonRead (
       if(InBuf->IsEmpty(InBuf)) {\r
         NumRead = filp->f_ops->fo_read(filp, &filp->f_offset, sizeof(wchar_t), &InChar);\r
         if(NumRead > 0) {\r
-          Status = InBuf->Write(InBuf, &InChar, 1);  // Buffer the character\r
+          (void) InBuf->Write(InBuf, &InChar, 1);  // Buffer the character\r
         }\r
       }\r
   //    break;\r
index 9cff086c6ad10108f3f8356a33ced1ddeb671d7a..31d2acd6291561b0875e9c0d257abf00076d83ce 100644 (file)
@@ -1,7 +1,6 @@
-/*\r
- * Copyright (c) 1999, 2000\r
- * Intel Corporation.\r
- * All rights reserved.\r
+/** @file\r
+ *\r
+ * Copyright (c) 1999 - 2014, Intel Corporation. All rights reserved.<BR>\r
  *\r
  * Redistribution and use in source and binary forms, with or without modification,\r
  * are permitted provided that the following conditions are met:\r
@@ -100,9 +99,11 @@ writev(
     int iovcnt\r
     )\r
 {\r
-  const struct iovec  *pVecTmp;\r
-  char      *pBuf, *pBufTmp;\r
-  size_t      TotalBytes, i, ret;\r
+  const struct iovec   *pVecTmp;\r
+  char                 *pBuf;\r
+  size_t                TotalBytes;\r
+  size_t                i;\r
+  size_t                ret;\r
 \r
   //\r
   //  See how much memory we'll need\r
@@ -126,7 +127,7 @@ writev(
   //  Copy vectors to the buffer\r
   //\r
 \r
-  for (pBufTmp = pBuf; iovcnt; iovcnt--) {\r
+  for (; iovcnt; iovcnt--) {\r
     bcopy(iov->iov_base, pBuf, iov->iov_len);\r
     pBuf += iov->iov_len;\r
     iov++;\r