Changeset 1068:cd86a0f4adab in roaraudio for libroar/socket.c

Timestamp:

12/28/08 02:40:49 (15 years ago)

Author:

phi

Branch:

default

Phase:

public

Message:

make OpenBSD security warnings patch a bit happyer...: strcpy() -> strncpy()

File:

• libroar/socket.c (modified) (2 diffs)

libroar/socket.c

@@ -346 +346 @@
  } else {
   bind_sockaddr.sdn_objnamel  = ROAR_dn_htons(strlen(object));
-  strcpy((char*)bind_sockaddr.sdn_objname, object); // FIXME: shouldn't we use strncpy()?
+  if ( bind_sockaddr.sdn_objnamel > DN_MAXOBJL )
+   bind_sockaddr.sdn_objnamel = DN_MAXOBJL;
+  strncpy((char*)bind_sockaddr.sdn_objname, object, DN_MAXOBJL);
  }
 
@@ -884 +886 @@
 
  if ( !strcmp(host, "+fork") ) {
-  strcpy(rcmd, "roard --no-listen --client-fh 0");
+  strncpy(rcmd, "roard --no-listen --client-fh 0", 32);
  } else {
   if ( use_socat ) {