RE: bzr commit into mysql-6.0-backup branch (cbell:2638) Bug#33364, Bug#35230, WL#4296

[Chuck Bell]

Oystein,

Just to let you know you aren't missing anything. There is indeed a bug in
this code which I have fixed. Golly. 

> >>  > +void Logger::report_engine(const char *engine_name)  > +{
> >>  > +  String str;    // engines string
> >>  > +  DBUG_ASSERT(m_state == READY || m_state == RUNNING); 
>  > +  > +  
> >> str.length(0);  > +  if (m_op_hist->engine_name &&
> >>  > +      (strlen(m_op_hist->engine_name) > 0) &&
> >>  > +      (strlen(engine_name) > 0))
> >>  > +    str.append(", ");
> >>  > +  if (strlen(engine_name) > 0)
> >>  > +    str.append(engine_name);
> >>  > +  if (str.length() > 0)
> >>  > +  {
> >>  > +    if (m_op_hist->engine_name)
> >>  > +      delete m_op_hist->engine_name;
> >>  > +    m_op_hist->engine_name = new char[str.length()+1];
> >>  > +    strcpy(m_op_hist->engine_name, str.ptr());
> >>  > +  }
> >>  > +}

New patch later this week...fixing lots and lots of things.

Chuck