~mathiaz/ubuntu/lucid/mysql-dfsg-5.1/zap-bug-552053

« back to all changes in this revision

Viewing changes to storage/federated/ha_federated.cc

  • Committer: Bazaar Package Importer
  • Author(s): Mathias Gug
  • Date: 2009-06-25 12:55:45 UTC
  • mfrom: (1.1.2 upstream) (0.1.3 experimental)
  • Revision ID: james.westby@ubuntu.com-20090625125545-m8ogs96zzsri74xe
Tags: 5.1.34-1ubuntu1
* Merge from debian experimental (and 5.0 from main), remaining changes:
  - debian/mysql-server-5.1.config:
    + ask for MySQL root password at priority high instead of medium so
      that the password prompt is seen on a default install. (LP: #319843)
    + don't ask for root password when upgrading from a 5.0 install.
  - debian/control:
    + Make libmysqlclient16-dev a transitional package depending on
      libmysqlclient-dev.
    + Make libmysqlclient-dev conflict with libmysqlclient15-dev.
    + Don't build mysql-server, mysql-client, mysql-common and
      libmysqlclient15-dev binary packages since they're still provided
      by mysql-dfsg-5.0.
    + Make mysql-{client,server}-5.1 packages conflict and
      replace mysql-{client,server}-5.0, but not provide
      mysql-{client,server}.
    + Depend on a specific version of mysql-common rather than the src
      version of mysql-dfsg-5.1 since mysql-common is currently part of
      mysql-dfsg-5.0.
    + Lower mailx from a Recommends to a Suggests to avoid pulling in
      a full MTA on all installs of mysql-server. (LP: #259477)
  - debian/rules:
    + added -fno-strict-aliasing to CFLAGS to get around mysql testsuite
      build failures.
    + install mysql-test and sql-bench to /usr/share/mysql/ rather than
      /usr/.
  - debian/additions/debian-start.inc.sh: support ANSI mode (LP: #310211)
  - Add AppArmor profile:
    - debian/apparmor-profile: apparmor profile.
    - debian/rules, debian/mysql-server-5.0.files: install apparmor profile.
    - debian/mysql-server-5.0.dirs: add etc/apparmor.d/force-complain
    - debian/mysql-server-5.0.postrm: remove symlink in force-complain/ on
      purge.
    - debian/mysql-server-5.1.README.Debian: add apparmor documentation.
    - debian/additions/my.cnf: Add warning about apparmor. (LP: #201799)
    - debian/mysql-server-5.1.postinst: reload apparmor profiles.
  - debian/additions/my.cnf: remove language option. Error message files are
    located in a different directory in MySQL 5.0. Setting the language
    option to use /usr/share/mysql/english breaks 5.0. Both 5.0 and 5.1
    use a default value that works. (LP: #316974)
  - debian/mysql-server-5.1.mysql.init:
    + Clearly indicate that we do not support running multiple instances
      of mysqld by duplicating the init script.
      (closes: #314785, #324834, #435165, #444216)
    + Properly parameterize all existing references to the mysql config
      file (/etc/mysql/my.cnf).
  - debian/mysql-server-5.0.postinst: Clear out the second password
    when setting up mysql. (LP: #344816)
  - mysql-server-core-5.1 package for files needed by Akonadi:
    + debian/control: create mysql-server-core-5.1 package.
    + debian/mysql-server-core-5.1.files, debian/mysql-server-5.1.files:
      move core mysqld files to mysql-server-core-5.1 package.
  - Don't package sql-bench and mysql-test file.
* Dropped changes:
  - debian/patches/92_ssl_test_cert.dpatch: certificate expiration in
    test suite (LP: #323755). Included upstream.
* Dropped from 5.0:
  - apparmor profile:
    - debian/control: Recommends apparmor >= 2.1+1075-0ubuntu6. All version
      of apparmor-profile (>hardy) are higher than this version.
    - debian/mysql-server-5.0.preinst: create symlink for force-complain/
      on pre-feisty upgrades, upgrades where apparmor-profiles profile is
      unchanged (ie non-enforcing) and upgrades where the profile
      doesn't exist. Support for pre-hardy upgrades is no longer needed.
* debian/mysql-server-5.1.postinst: fix debian-sys-maint user creation.

Show diffs side-by-side

added added

removed removed

Lines of Context:
505
505
        in sql_show.cc except that quoting always occurs.
506
506
*/
507
507
 
508
 
static bool append_ident(String *string, const char *name, uint length,
 
508
static bool append_ident(String *string, const char *name, size_t length,
509
509
                         const char quote_char)
510
510
{
511
511
  bool result;
515
515
 
516
516
  if (quote_char)
517
517
  {
518
 
    string->reserve(length * 2 + 2);
 
518
    string->reserve((uint) length * 2 + 2);
519
519
    if ((result= string->append(&quote_char, 1, system_charset_info)))
520
520
      goto err;
521
521
 
533
533
    result= string->append(&quote_char, 1, system_charset_info);
534
534
  }
535
535
  else
536
 
    result= string->append(name, length, system_charset_info);
 
536
    result= string->append(name, (uint) length, system_charset_info);
537
537
 
538
538
err:
539
539
  DBUG_RETURN(result);
543
543
static int parse_url_error(FEDERATED_SHARE *share, TABLE *table, int error_num)
544
544
{
545
545
  char buf[FEDERATED_QUERY_BUFFER_SIZE];
546
 
  int buf_len;
 
546
  size_t buf_len;
547
547
  DBUG_ENTER("ha_federated parse_url_error");
548
548
 
549
549
  buf_len= min(table->s->connect_string.length,
721
721
    {
722
722
      share->connection_string[share->table_name - share->connection_string]= '\0';
723
723
      share->table_name++;
724
 
      share->table_name_length= strlen(share->table_name);
 
724
      share->table_name_length= (uint) strlen(share->table_name);
725
725
 
726
726
      DBUG_PRINT("info", 
727
727
                 ("internal format, parsed table_name share->connection_string \
1489
1489
  pthread_mutex_lock(&federated_mutex);
1490
1490
 
1491
1491
  tmp_share.share_key= table_name;
1492
 
  tmp_share.share_key_length= strlen(table_name);
 
1492
  tmp_share.share_key_length= (uint) strlen(table_name);
1493
1493
  if (parse_url(&mem_root, &tmp_share, table, 0))
1494
1494
    goto error;
1495
1495
 
2161
2161
  {
2162
2162
    if (bitmap_is_set(table->write_set, (*field)->field_index))
2163
2163
    {
2164
 
      uint field_name_length= strlen((*field)->field_name);
 
2164
      size_t field_name_length= strlen((*field)->field_name);
2165
2165
      append_ident(&update_string, (*field)->field_name, field_name_length,
2166
2166
                   ident_quote_char);
2167
2167
      update_string.append(STRING_WITH_LEN(" = "));
2187
2187
 
2188
2188
    if (bitmap_is_set(table->read_set, (*field)->field_index))
2189
2189
    {
2190
 
      uint field_name_length= strlen((*field)->field_name);
 
2190
      size_t field_name_length= strlen((*field)->field_name);
2191
2191
      append_ident(&where_string, (*field)->field_name, field_name_length,
2192
2192
                   ident_quote_char);
2193
2193
      if (field_in_record_is_null(table, *field, (char*) old_data))
3158
3158
}
3159
3159
 
3160
3160
 
3161
 
int ha_federated::real_query(const char *query, uint length)
 
3161
int ha_federated::real_query(const char *query, size_t length)
3162
3162
{
3163
3163
  int rc= 0;
3164
3164
  DBUG_ENTER("ha_federated::real_query");
3169
3169
  if (!query || !length)
3170
3170
    goto end;
3171
3171
 
3172
 
  rc= mysql_real_query(mysql, query, length);
 
3172
  rc= mysql_real_query(mysql, query, (uint) length);
3173
3173
  
3174
3174
end:
3175
3175
  DBUG_RETURN(rc);