Messages in this thread Patch in this message | | | From | Michael Estner <> | Subject | [PATCH] avoid crashing the kernel | Date | Sat, 25 Sep 2021 22:04:30 +0200 |
| |
To avoid chrashing the kernel I use WARN_ON instead.
Signed-off-by: Michael Estner <michaelestner@web.de> --- drivers/staging/most/i2c/i2c.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/most/i2c/i2c.c b/drivers/staging/most/i2c/i2c.c index 7042f10887bb..e1edd892f9fd 100644 --- a/drivers/staging/most/i2c/i2c.c +++ b/drivers/staging/most/i2c/i2c.c @@ -68,7 +68,7 @@ static int configure_channel(struct most_interface *most_iface, struct hdm_i2c *dev = to_hdm(most_iface); unsigned int delay, pr;
- BUG_ON(ch_idx < 0 || ch_idx >= NUM_CHANNELS); + WARN_ON(ch_idx < 0 || ch_idx >= NUM_CHANNELS);
if (channel_config->data_type != MOST_CH_CONTROL) { pr_err("bad data type for channel %d\n", ch_idx); @@ -122,7 +122,7 @@ static int enqueue(struct most_interface *most_iface, struct hdm_i2c *dev = to_hdm(most_iface); int ret;
- BUG_ON(ch_idx < 0 || ch_idx >= NUM_CHANNELS); + WARN_ON(ch_idx < 0 || ch_idx >= NUM_CHANNELS);
if (ch_idx == CH_RX) { /* RX */ @@ -167,7 +167,7 @@ static int poison_channel(struct most_interface *most_iface, struct hdm_i2c *dev = to_hdm(most_iface); struct mbo *mbo;
- BUG_ON(ch_idx < 0 || ch_idx >= NUM_CHANNELS); + WARN_ON(ch_idx < 0 || ch_idx >= NUM_CHANNELS);
if (ch_idx == CH_RX) { if (!polling_rate) -- 2.25.1
| |