[haiku-commits] Re: haiku: hrev43576 - src/system/libroot/os

  • From: marcus overhagen <marcusoverhagen@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Fri, 30 Dec 2011 05:13:02 +0100

On Thu, Dec 29, 2011 at 8:16 PM,  <stpere@xxxxxxxxx> wrote:

> d2a423e: driver_settings: Replace type of bufferSize from size_t to ssize_t
>
>  At various locations, checks are made so that bufferSize is >= 0, which never
>  occurs. CID 4197.
> -put_string(char **_buffer, size_t *_bufferSize, char *string)
> +put_string(char **_buffer, ssize_t *_bufferSize, char *string)

I don't think this change is correct. whats the use in having a signed
buffer size?

Marcus

Other related posts: