Crash in Init() in memcached.hpp

Bug #576119 reported by Nono
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
libmemcached
Fix Released
Undecided
Unassigned

Bug Description

Memcache CopyConstructor and =operator could crash when calling init(), because servers_list could be empty, and memcached_servers_parse(servers_list.c_str()) returns NULL, then crashing inmemcached_server_free(servers):

CRASHES
<<<
void init()
  {
    memcached_server_st *servers;
    servers= memcached_servers_parse(servers_list.c_str());
    memcached_server_push(&memc, servers);
    memcached_server_free(servers);
  }
>>>

a simple if() will FIX that:
<<<
void init()
  {
    memcached_server_st *servers;
    servers= memcached_servers_parse(servers_list.c_str());
    if (servers != NULL)
    {
        memcached_server_push(&memc, servers);
        memcached_server_free(servers);
    }
  }
>>>

Brian Aker (brianaker)
Changed in libmemcached:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.