Remove use of buffer_insert from filters, MACs, stream ciphers

src/lib/filters/b64_filt.cpp

@@ -68,7 +68,9 @@ void Base64_Encoder::do_output(const uint8_t input[], size_t length) {
 * Convert some data into Base64
 */
 void Base64_Encoder::write(const uint8_t input[], size_t length) {
-   buffer_insert(m_in, m_position, input, length);
+   const size_t initial_fill = std::min(m_in.size() - m_position, length);
+   copy_mem(&m_in[m_position], input, initial_fill);
+
    if(m_position + length >= m_in.size()) {
       encode_and_send(m_in.data(), m_in.size());
       input += (m_in.size() - m_position);

src/lib/filters/hex_filt.cpp

@@ -64,7 +64,9 @@ void Hex_Encoder::encode_and_send(const uint8_t block[], size_t length) {
 * Convert some data into hex format
 */
 void Hex_Encoder::write(const uint8_t input[], size_t length) {
-   buffer_insert(m_in, m_position, input, length);
+   const size_t initial_fill = std::min(m_in.size() - m_position, length);
+   copy_mem(&m_in[m_position], input, initial_fill);
+
    if(m_position + length >= m_in.size()) {
       encode_and_send(m_in.data(), m_in.size());
       input += (m_in.size() - m_position);

src/lib/mac/cmac/cmac.cpp

@@ -20,7 +20,9 @@ namespace Botan {
 void CMAC::add_data(std::span<const uint8_t> input) {
    const size_t bs = output_length();
 
-   buffer_insert(m_buffer, m_position, input.data(), input.size());
+   const size_t initial_fill = std::min(m_buffer.size() - m_position, input.size());
+   copy_mem(m_buffer.data() + m_position, input.data(), initial_fill);
+
    if(m_position + input.size() > bs) {
       xor_buf(m_state, m_buffer, bs);
       m_cipher->encrypt(m_state);

src/lib/mac/poly1305/poly1305.cpp

@@ -184,7 +184,8 @@ void Poly1305::add_data(std::span<const uint8_t> input) {
    assert_key_material_set();
 
    if(m_buf_pos) {
-      buffer_insert(m_buf, m_buf_pos, input.data(), input.size());
+      const size_t initial_fill = std::min(m_buf.size() - m_buf_pos, input.size());
+      copy_mem(m_buf.data() + m_buf_pos, input.data(), initial_fill);
 
       if(m_buf_pos + input.size() >= m_buf.size()) {
          poly1305_blocks(m_poly, m_buf.data(), 1);
@@ -201,7 +202,8 @@ void Poly1305::add_data(std::span<const uint8_t> input) {
    }
 
    const auto remaining = in.take(in.remaining());
-   buffer_insert(m_buf, m_buf_pos, remaining.data(), remaining.size());
+   BOTAN_ASSERT_NOMSG(m_buf_pos + remaining.size() < m_buf.size());
+   copy_mem(m_buf.data() + m_buf_pos, remaining.data(), remaining.size());
    m_buf_pos += remaining.size();
 }
 

src/lib/stream/ctr/ctr.cpp

@@ -153,7 +153,7 @@ void CTR_BE::set_iv_bytes(const uint8_t iv[], size_t iv_len) {
 
    m_iv.resize(m_block_size);
    zeroise(m_iv);
-   buffer_insert(m_iv, 0, iv, iv_len);
+   copy_mem(&m_iv[0], iv, iv_len);
 
    seek(0);
 }
@@ -211,7 +211,8 @@ void CTR_BE::seek(uint64_t offset) {
    const uint64_t base_counter = m_ctr_blocks * (offset / m_counter.size());
 
    zeroise(m_counter);
-   buffer_insert(m_counter, 0, m_iv);
+   BOTAN_ASSERT_NOMSG(m_counter.size() >= m_iv.size());
+   copy_mem(&m_counter[0], &m_iv[0], m_iv.size());
 
    const size_t BS = m_block_size;
 
@@ -239,7 +240,7 @@ void CTR_BE::seek(uint64_t offset) {
    } else {
       // do everything sequentially:
       for(size_t i = 1; i != m_ctr_blocks; ++i) {
-         buffer_insert(m_counter, i * BS, &m_counter[(i - 1) * BS], BS);
+         copy_mem(&m_counter[i * BS], &m_counter[(i - 1) * BS], BS);
 
          for(size_t j = 0; j != m_ctr_size; ++j) {
             if(++m_counter[i * BS + (BS - 1 - j)]) {

src/lib/stream/ofb/ofb.cpp

@@ -75,7 +75,8 @@ void OFB::set_iv_bytes(const uint8_t iv[], size_t iv_len) {
    }
 
    zeroise(m_buffer);
-   buffer_insert(m_buffer, 0, iv, iv_len);
+   BOTAN_ASSERT_NOMSG(m_buffer.size() >= iv_len);
+   copy_mem(&m_buffer[0], iv, iv_len);
 
    m_cipher->encrypt(m_buffer);
    m_buf_pos = 0;