<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - add sanity check to avoid buffer overflow"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=102690#c1">Comment # 1</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - add sanity check to avoid buffer overflow"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=102690">bug 102690</a>
              from <span class="vcard"><a class="email" href="mailto:ppaalanen@gmail.com" title="Pekka Paalanen <ppaalanen@gmail.com>"> <span class="fn">Pekka Paalanen</span></a>
</span></b>
        <pre>Hi,

thanks for the patch.

Patches should be sent to the wayland-devel@ mailing list as per the
contribution instructions linked to from <a href="https://wayland.freedesktop.org/">https://wayland.freedesktop.org/</a> .

I believe the check you are adding is redundant with the checks in the callers
of wl_buffer_put(): wl_connection_write() and wl_connection_queue(). Both
callers first check if the 'count' bytes fit in wl_buffer_size() and if not,
they force a flush that either completely empties the buffer or fails and
causes an early exit from the functions. Therefore the callers ensure that
either the 'count' bytes fit, or the buffer is completely empty.

But I do agree that all that looks fragile.

Even if the check you add seems redundant, I think it would be good to have. A
sanity check, as you say.

Please, re-send this patch to wayland-devel@ mailing list with a link to this
bug report and the following line:

Acked-by: Pekka Paalanen <<a href="mailto:pekka.paalanen@collabora.co.uk">pekka.paalanen@collabora.co.uk</a>>

added to the commit message.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>