tree 26a25778bdc503c3882fce537cd7dd61c3f27738
parent 8b3fc2d9ce09fac1c3d172bea20043b1e201adf4
author George Burgess IV <gbiv@google.com> 1758557877 -0600
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> 1758570293 -0700

CHROMIUM: restore `optlength == 0` check

The chromium-specific http://crrev.com/c/1709531 removed this check
(& was cherry-picked to the current branch by
http://crrev.com/c/4588053). Enabling Clang's local-bounds sanitizer
catches that we're doing:

```
optptr = malloc(0);
*optptr = '\0';
```

While this behavior is implementation-defined, it's definitely a code
smell; just `malloc(1)` instead.

BUG=b:431006422
TEST=`emerge-amd64-generic net-print/cups` passes with new bounds
TEST=sanitizer.

Change-Id: I0f9e6f128862251a0febcb2032eaed4d06e2d20d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/cups/+/6972769
Commit-Queue: George Burgess <gbiv@chromium.org>
Reviewed-by: Benjamin Gordon <bmgordon@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>