[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/7] hw/i2c: pmbus: block uninitialised string reads
From: |
Corey Minyard |
Subject: |
Re: [PATCH 4/7] hw/i2c: pmbus: block uninitialised string reads |
Date: |
Fri, 31 Mar 2023 08:53:04 -0500 |
On Fri, Mar 31, 2023 at 12:07:53AM +0000, Titus Rwantare wrote:
> Devices models calling pmbus_send_string can't be relied upon to
> send a non-zero pointer. This logs an error and doesn't segfault.
>
> Reviewed-by: Patrick Venture <venture@google.com>
> Signed-off-by: Titus Rwantare <titusr@google.com>
Acked-by: Corey Minyard <cminyard@mvista.com>
> ---
> hw/i2c/pmbus_device.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/hw/i2c/pmbus_device.c b/hw/i2c/pmbus_device.c
> index bb42e410b4..18e629eaac 100644
> --- a/hw/i2c/pmbus_device.c
> +++ b/hw/i2c/pmbus_device.c
> @@ -94,6 +94,13 @@ void pmbus_send64(PMBusDevice *pmdev, uint64_t data)
>
> void pmbus_send_string(PMBusDevice *pmdev, const char *data)
> {
> + if (!data) {
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: %s: uninitialised read from 0x%02x\n",
> + __func__, DEVICE(pmdev)->canonical_path, pmdev->code);
> + return;
> + }
> +
> size_t len = strlen(data);
> g_assert(len + pmdev->out_buf_len < SMBUS_DATA_MAX_LEN);
> pmdev->out_buf[len + pmdev->out_buf_len] = len;
> --
> 2.40.0.423.gd6c402a77b-goog
>
- Re: [PATCH 3/7] hw/i2c: pmbus: add fan support, (continued)
- [PATCH 2/7] hw/i2c: pmbus: add vout mode bitfields, Titus Rwantare, 2023/03/30
- [PATCH 7/7] tests/qtest: add tests for ADM1266, Titus Rwantare, 2023/03/30
- [PATCH 6/7] hw/sensor: add ADM1266 device model, Titus Rwantare, 2023/03/30
- [PATCH 5/7] hw/i2c: pmbus: add VCAP register, Titus Rwantare, 2023/03/30
- [PATCH 4/7] hw/i2c: pmbus: block uninitialised string reads, Titus Rwantare, 2023/03/30
- Re: [PATCH 4/7] hw/i2c: pmbus: block uninitialised string reads,
Corey Minyard <=