dbusbind.c patch, fixes possible segfault

[David Hansen <david.hansen@gmx.net>]

[-- Attachment #1: Type: text/plain, Size: 288 bytes --]


This one is similar to the previous dbusbind.c patch.  No need to copy
the string and if dbus_bus_get_unique_name () returns NULL strcpy will
segfault.

David

2008-04-13  David Hansen  <david.hansen@gmx.net>

	* dbusbind.c (dbus-get-unique-name): Remove extra copying of name
	string.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Type: text/x-patch, Size: 949 bytes --]

*** dbusbind.c.~1.24.~	2008-04-05 23:46:55.000000000 +0200
--- dbusbind.c	2008-04-13 15:19:27.000000000 +0200
***************
*** 701,707 ****
       Lisp_Object bus;
  {
    DBusConnection *connection;
!   char name[DBUS_MAXIMUM_NAME_LENGTH];
  
    /* Check parameters.  */
    CHECK_SYMBOL (bus);
--- 701,707 ----
       Lisp_Object bus;
  {
    DBusConnection *connection;
!   const char *name;
  
    /* Check parameters.  */
    CHECK_SYMBOL (bus);
***************
*** 710,716 ****
    connection = xd_initialize (bus);
  
    /* Request the name.  */
!   strcpy (name, dbus_bus_get_unique_name (connection));
    if (name == NULL)
      xsignal1 (Qdbus_error, build_string ("No unique name available"));
  
--- 710,716 ----
    connection = xd_initialize (bus);
  
    /* Request the name.  */
!   name = dbus_bus_get_unique_name (connection);
    if (name == NULL)
      xsignal1 (Qdbus_error, build_string ("No unique name available"));