Android 5.1.1 release 9
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iEYEABECAAYFAlW6dFwACgkQ6K0/gZqxDnigPwCbBK1F91Vd2LDRNR8jyZrm3BrR
Hg0AniPzU4rnv7VA2C1tdWv9GBKWOImE
=Zb+c
-----END PGP SIGNATURE-----
Fun with buffer overrruns.

In get_option(): don't read past the end of the option buffer.

Also add a small unittest to verify sane behaviour for the above.
The dhcpcd code is not easily refactored into a library, nor is it
entirely possible to include some header files directly since some
structures use C++ reserved keywords ("new") for variable names.

In print_option(): use of snprintf() returns the length that
/would/ have been written.  Add checks that the output buffer
is not overrun when printing.

Bug: 18356137
Bug: 18356135
Change-Id: I0f907b8a952208749226ba034a416d773e068f8a
3 files changed