Преобразовать функцию XOR, чтобы она возвращала char*, а не строку

РЕДАКТИРОВАТЬ: это было ошибкой, но исправлено с помощью @JonathanGrynspan и @john, и теперь моя окончательная версия:

Я знаю, что это не красиво, но делает то, что вы хотите:

char* XOR(string s, int key){
 string res = "";
 for (size_t temp = 0; temp < s.size(); ++temp){
  res += s[temp] ^ ((key + temp) % 255);
 }
 char * cstr = new char [res.length()+1];
 copy(res.begin(),res.end(),cstr);
 return cstr;
}

Является ли ваша входная строка перезаписываемой? Если это так, это самый простой и, вероятно, самый быстрый способ:

void XOR( char *input, int key ){
    for(int temp = 0; input[temp] != '\0'; ++temp){
        input[temp] ^= (key + temp) % 255;
    }
}

Он изменяет заданную строку вместо того, чтобы возвращать что-то новое. Но поскольку есть все шансы, что вы получите \0 символов в выводе, вы можете найти это более надежным:

void XOR( char *input, int key, int length ){
    for(int temp = 0; temp < length; ++temp){
        input[temp] ^= (key + temp) % 255;
    }
}

(очевидно, длина не изменилась)

Обновлять:

Увидев комментарии, возможно, это то, что вы хотите. Но будьте с ним осторожны; вам нужно будет помнить об освобождении возвращаемой памяти, к чему не привыкли программисты, использующие только C++;

char *XOR( const string &s, int key ) {
    const std::size_t l = s.size( );
    char *r = (char *) malloc( (l + 1) * sizeof( char ) );
    for( std::size_t i = 0; i < l; ++ i ) {
        r[i] = s[i] ^ ((key + (int) i) % 255)
    }
    r[l] = '\0'; // only needed if you need a null-capped string, which seems unlikely since your string could well contain nulls anyway
    return r;
}