fwiw, this made me cry. a lot. grr.


-adrian


On 2 June 2016 at 22:01, Adrian Chadd <adr...@freebsd.org> wrote:
> Author: adrian
> Date: Fri Jun  3 05:01:35 2016
> New Revision: 301236
> URL: https://svnweb.freebsd.org/changeset/base/301236
>
> Log:
>   [drm] fix up hex_dump_to_buffer to not overflow linebuf.
>
>   That check wasn't enough to handle appending a two byte character
>   following it.
>
>   This prevented my T400 (Intel Core 2 Duo P8400) from attaching;
>   it would panic from a stack overflow detection.
>
> Modified:
>   head/sys/dev/drm2/drm_os_freebsd.c
>
> Modified: head/sys/dev/drm2/drm_os_freebsd.c
> ==============================================================================
> --- head/sys/dev/drm2/drm_os_freebsd.c  Fri Jun  3 03:40:39 2016        
> (r301235)
> +++ head/sys/dev/drm2/drm_os_freebsd.c  Fri Jun  3 05:01:35 2016        
> (r301236)
> @@ -422,7 +422,7 @@ hex_dump_to_buffer(const void *buf, size
>                         }
>                 }
>
> -               if (j > linebuflen - 1)
> +               if (j > linebuflen - 4)
>                         break;
>
>                 sprintf(linebuf + j, "%02X", c);
>
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to