Possible race in nsc-ircc.ko

2017-08-25 Thread Anton Volkov
e dongle with id 0 exists). Is this case feasible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible race in nsc-ircc.ko

2017-08-25 Thread Anton Volkov
e dongle with id 0 exists). Is this case feasible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible memory leak in cafe_ccic.ko

2017-08-22 Thread Anton Volkov
m_v4l_release() freed and poisoned them. Is this feasible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible memory leak in cafe_ccic.ko

2017-08-22 Thread Anton Volkov
m_v4l_release() freed and poisoned them. Is this feasible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible double free in iowarrior.ko

2017-08-22 Thread Anton Volkov
n both 'if' branches? Thank you for your time -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible double free in iowarrior.ko

2017-08-22 Thread Anton Volkov
n both 'if' branches? Thank you for your time -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

[PATCH] tegra114: fix to a race condition due to early registration of interrupt handler

2017-08-22 Thread Anton Volkov
ace infeasible. Found by by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton Volkov <avol...@ispras.ru> --- drivers/spi/spi-tegra114.c | 31 +++ 1 file changed, 15 insertions(+), 16 deletions(-) diff --git a/drivers/spi/spi-tegra114.c

[PATCH] tegra114: fix to a race condition due to early registration of interrupt handler

2017-08-22 Thread Anton Volkov
ace infeasible. Found by by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton Volkov --- drivers/spi/spi-tegra114.c | 31 +++ 1 file changed, 15 insertions(+), 16 deletions(-) diff --git a/drivers/spi/spi-tegra114.c b/drivers/spi/spi-tegra11

Possible bug in cypress_m8.ko

2017-08-22 Thread Anton Volkov
r your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible bug in cypress_m8.ko

2017-08-22 Thread Anton Volkov
r your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible race in mlx5_ib.ko

2017-08-18 Thread Anton Volkov
concurrently with the same ent then there is a possibility of a race between the accesses to ent->cur. In worst case in limit_write new keys wouldn't be added. Is it feasible from your point of view? If so, is it a benign race or a serious one? Thank you for your time. -- Anton Volkov Linux Ver

Possible race in mlx5_ib.ko

2017-08-18 Thread Anton Volkov
concurrently with the same ent then there is a possibility of a race between the accesses to ent->cur. In worst case in limit_write new keys wouldn't be added. Is it feasible from your point of view? If so, is it a benign race or a serious one? Thank you for your time. -- Anton Volkov Linux Ver

Possible race in xilinx-xadc.ko

2017-08-18 Thread Anton Volkov
mask though it should probably be different. Is this race feasible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible race in xilinx-xadc.ko

2017-08-18 Thread Anton Volkov
mask though it should probably be different. Is this race feasible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Re: Possible null pointer dereference in adutux.ko

2017-08-18 Thread Anton Volkov
On 15.08.2017 18:58, Oliver Neukum wrote: Am Dienstag, den 15.08.2017, 16:38 +0300 schrieb Anton Volkov: On 15.08.2017 16:20, Oliver Neukum wrote: Am Dienstag, den 15.08.2017, 15:59 +0300 schrieb Anton Volkov: Hello. While searching for races in the Linux kernel I've come across "dr

Re: Possible null pointer dereference in adutux.ko

2017-08-18 Thread Anton Volkov
On 15.08.2017 18:58, Oliver Neukum wrote: Am Dienstag, den 15.08.2017, 16:38 +0300 schrieb Anton Volkov: On 15.08.2017 16:20, Oliver Neukum wrote: Am Dienstag, den 15.08.2017, 15:59 +0300 schrieb Anton Volkov: Hello. While searching for races in the Linux kernel I've come across "dr

Possible race in ibmasm.ko

2017-08-18 Thread Anton Volkov
our point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible race in ibmasm.ko

2017-08-18 Thread Anton Volkov
our point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible race in ks8851_mll.ko

2017-08-15 Thread Anton Volkov
them changes the ks->cmd_reg_cache it is possible that both will use the same value though it should be different. Is this race feasible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible race in ks8851_mll.ko

2017-08-15 Thread Anton Volkov
them changes the ks->cmd_reg_cache it is possible that both will use the same value though it should be different. Is this race feasible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible race in via-ircc.ko

2017-08-15 Thread Anton Volkov
om your point of view? If it is feasible, is it a benign race or something serious? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible race in via-ircc.ko

2017-08-15 Thread Anton Volkov
om your point of view? If it is feasible, is it a benign race or something serious? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible race in ucb1400_ts.ko

2017-08-15 Thread Anton Volkov
_irq() The value of ucb->stopped may be changed in the midst of 'while' loop iterations or prevent all of them from happening. Is this feasible from your point of view? If so, is it a benign race or is it serious? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS

Possible race in ucb1400_ts.ko

2017-08-15 Thread Anton Volkov
_irq() The value of ucb->stopped may be changed in the midst of 'while' loop iterations or prevent all of them from happening. Is this feasible from your point of view? If so, is it a benign race or is it serious? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS

Re: Possible null pointer dereference in adutux.ko

2017-08-15 Thread Anton Volkov
On 15.08.2017 16:20, Oliver Neukum wrote: Am Dienstag, den 15.08.2017, 15:59 +0300 schrieb Anton Volkov: Hello. While searching for races in the Linux kernel I've come across "drivers/usb/misc/adutux.ko" module. Here is a question that I came up with while analyzing results. Lines

Re: Possible null pointer dereference in adutux.ko

2017-08-15 Thread Anton Volkov
On 15.08.2017 16:20, Oliver Neukum wrote: Am Dienstag, den 15.08.2017, 15:59 +0300 schrieb Anton Volkov: Hello. While searching for races in the Linux kernel I've come across "drivers/usb/misc/adutux.ko" module. Here is a question that I came up with while analyzing results. Lines

Possible race in c4.ko

2017-08-15 Thread Anton Volkov
signed an undesirable value and wrong controller may handle messages. Is this case feasible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible race in c4.ko

2017-08-15 Thread Anton Volkov
signed an undesirable value and wrong controller may handle messages. Is this case feasible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible null pointer dereference in adutux.ko

2017-08-15 Thread Anton Volkov
s case feasible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible null pointer dereference in adutux.ko

2017-08-15 Thread Anton Volkov
s case feasible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Re: [PATCH] loop: fix to a race condition due to the early registration of device

2017-08-10 Thread Anton Volkov
Hello, Omar. It was a purely theoretical race that had been considered to be possible in real-life. Regards, Anton On 09.08.2017 01:00, Omar Sandoval wrote: On Mon, Aug 07, 2017 at 03:37:50PM +0300, Anton Volkov wrote: The early device registration made possible a race leading

Re: [PATCH] loop: fix to a race condition due to the early registration of device

2017-08-10 Thread Anton Volkov
Hello, Omar. It was a purely theoretical race that had been considered to be possible in real-life. Regards, Anton On 09.08.2017 01:00, Omar Sandoval wrote: On Mon, Aug 07, 2017 at 03:37:50PM +0300, Anton Volkov wrote: The early device registration made possible a race leading

Re: Question about apds990x.ko

2017-08-10 Thread Anton Volkov
, 2017 at 05:57:30PM +0300, Anton Volkov wrote: Hello. While searching for races in the Linux kernel I've come across "drivers/misc/apds990x.ko" module. Here are questions that I came up with while analyzing results. Lines are given using the info from Linux v4.12. Consider the foll

Re: Question about apds990x.ko

2017-08-10 Thread Anton Volkov
, 2017 at 05:57:30PM +0300, Anton Volkov wrote: Hello. While searching for races in the Linux kernel I've come across "drivers/misc/apds990x.ko" module. Here are questions that I came up with while analyzing results. Lines are given using the info from Linux v4.12. Consider the foll

Question about apds990x.ko

2017-08-10 Thread Anton Volkov
ension status? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Question about apds990x.ko

2017-08-10 Thread Anton Volkov
ension status? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

[PATCH] tegra114: fix to a race condition due to early registration of interrupt handler

2017-08-10 Thread Anton Volkov
ace infeasible. Found by by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton Volkov <avol...@ispras.ru> --- drivers/spi/spi-tegra114.c | 31 +++ 1 file changed, 15 insertions(+), 16 deletions(-) diff --git a/drivers/spi/spi-tegra114.c

[PATCH] tegra114: fix to a race condition due to early registration of interrupt handler

2017-08-10 Thread Anton Volkov
ace infeasible. Found by by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton Volkov --- drivers/spi/spi-tegra114.c | 31 +++ 1 file changed, 15 insertions(+), 16 deletions(-) diff --git a/drivers/spi/spi-tegra114.c b/drivers/spi/spi-tegra11

Question about via-ircc.ko

2017-08-08 Thread Anton Volkov
om your point of view? If it is feasible, is it a benign race or something serious? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Question about via-ircc.ko

2017-08-08 Thread Anton Volkov
om your point of view? If it is feasible, is it a benign race or something serious? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Question about bh1770glc.ko

2017-08-08 Thread Anton Volkov
. during the bh1770_resume. Do you know anything about whether this is intended or this is a bug? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Question about bh1770glc.ko

2017-08-08 Thread Anton Volkov
. during the bh1770_resume. Do you know anything about whether this is intended or this is a bug? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible null pointer dereference in rcar-dmac.ko

2017-08-08 Thread Anton Volkov
e. Thus there might be a NULL pointer dereference in rcar_dmac_chan_start_xfer while accessing chan->chan.device->dev which is equal to (>engine)->dev. Is this possible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible null pointer dereference in rcar-dmac.ko

2017-08-08 Thread Anton Volkov
e. Thus there might be a NULL pointer dereference in rcar_dmac_chan_start_xfer while accessing chan->chan.device->dev which is equal to (>engine)->dev. Is this possible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible null pointer dereference in isp1760.ko

2017-08-07 Thread Anton Volkov
0_udc_init_eps and list_empty() tries to access the '->next' pointer member of the passed parameter. Is this case possible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible null pointer dereference in isp1760.ko

2017-08-07 Thread Anton Volkov
0_udc_init_eps and list_empty() tries to access the '->next' pointer member of the passed parameter. Is this case possible from your point of view? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible race in pc87413_wdt.ko

2017-08-07 Thread Anton Volkov
device registration a bit further down in the pc87413_init function? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible race in pc87413_wdt.ko

2017-08-07 Thread Anton Volkov
device registration a bit further down in the pc87413_init function? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Re: [PATCH] loop: fix to a race condition due to the early registration of device

2017-08-07 Thread Anton Volkov
This is more of a style-oriented suggestion. This kind of template is commonly used in other modules. Regards, Anton On 07.08.2017 15:54, Johannes Thumshirn wrote: On Mon, Aug 07, 2017 at 03:37:50PM +0300, Anton Volkov wrote: +err_out: return err; Any reason you can't just use

Re: [PATCH] loop: fix to a race condition due to the early registration of device

2017-08-07 Thread Anton Volkov
This is more of a style-oriented suggestion. This kind of template is commonly used in other modules. Regards, Anton On 07.08.2017 15:54, Johannes Thumshirn wrote: On Mon, Aug 07, 2017 at 03:37:50PM +0300, Anton Volkov wrote: +err_out: return err; Any reason you can't just use

[PATCH v2] hysdn: fix to a race condition in put_log_buffer

2017-08-07 Thread Anton Volkov
won't be feasible in the future. Found by by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton Volkov <avol...@ispras.ru> --- v2: Fixed coding style issues drivers/isdn/hysdn/hysdn_proclog.c | 28 +--- 1 file changed, 13 insertions(+), 15 del

[PATCH v2] hysdn: fix to a race condition in put_log_buffer

2017-08-07 Thread Anton Volkov
won't be feasible in the future. Found by by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton Volkov --- v2: Fixed coding style issues drivers/isdn/hysdn/hysdn_proclog.c | 28 +--- 1 file changed, 13 insertions(+), 15 deletions(-) diff --git

[PATCH] loop: fix to a race condition due to the early registration of device

2017-08-07 Thread Anton Volkov
The early device registration made possible a race leading to allocations of disks with wrong minors. This patch moves the device registration further down the loop_init function to make the race infeasible. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton

[PATCH] loop: fix to a race condition due to the early registration of device

2017-08-07 Thread Anton Volkov
The early device registration made possible a race leading to allocations of disks with wrong minors. This patch moves the device registration further down the loop_init function to make the race infeasible. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton

[PATCH] loop: fix to a race condition due to the early registration of device

2017-08-03 Thread Anton Volkov
The early device registration made possible a race leading to allocations of disks with wrong minors. This patch moves the device registration further down the loop_init function to make the race infeasible. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton

[PATCH] loop: fix to a race condition due to the early registration of device

2017-08-03 Thread Anton Volkov
The early device registration made possible a race leading to allocations of disks with wrong minors. This patch moves the device registration further down the loop_init function to make the race infeasible. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton

[PATCH] hysdn: fix to a race condition in put_log_buffer

2017-08-03 Thread Anton Volkov
won't be feasible in the future. Found by by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton Volkov <avol...@ispras.ru> --- drivers/isdn/hysdn/hysdn_proclog.c | 27 --- 1 file changed, 12 insertions(+), 15 deletions(-) diff --git a/driver

[PATCH] hysdn: fix to a race condition in put_log_buffer

2017-08-03 Thread Anton Volkov
won't be feasible in the future. Found by by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton Volkov --- drivers/isdn/hysdn/hysdn_proclog.c | 27 --- 1 file changed, 12 insertions(+), 15 deletions(-) diff --git a/drivers/isdn/hysdn/hysdn_proclog.c

Possible race in loop.ko

2017-07-28 Thread Anton Volkov
or your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

Possible race in loop.ko

2017-07-28 Thread Anton Volkov
or your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru

[PATCH] hysdn: fix to a race condition in put_log_buffer

2017-07-28 Thread Anton Volkov
cases won't be feasible in the future. Found by by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton Volkov <avol...@ispras.ru> --- drivers/isdn/hysdn/hysdn_proclog.c | 27 --- 1 file changed, 12 insertions(+), 15 deletions(-) diff --git a/d

[PATCH] hysdn: fix to a race condition in put_log_buffer

2017-07-28 Thread Anton Volkov
cases won't be feasible in the future. Found by by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Anton Volkov --- drivers/isdn/hysdn/hysdn_proclog.c | 27 --- 1 file changed, 12 insertions(+), 15 deletions(-) diff --git a/drivers/isdn/hysdn

Possible race in hysdn.ko

2017-07-27 Thread Anton Volkov
Hello. While searching for races in the Linux kernel I've come across "drivers/isdn/hysdn/hysdn.ko" module. Here is a question that I came up with while analysing results. Lines are given using the info from Linux v4.12. In hysdn_proclog.c file in put_log_buffer function a non-standard type of

Possible race in hysdn.ko

2017-07-27 Thread Anton Volkov
Hello. While searching for races in the Linux kernel I've come across "drivers/isdn/hysdn/hysdn.ko" module. Here is a question that I came up with while analysing results. Lines are given using the info from Linux v4.12. In hysdn_proclog.c file in put_log_buffer function a non-standard type of

Possible race in spi-tegra114.ko

2017-07-24 Thread Anton Volkov
ter (spi-tegra114.c: line 1125) or a write to tspi->def_command1_reg (spi-tegra114.c: line 1121)? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru <mailto:avol...@ispras.ru>

Possible race in spi-tegra114.ko

2017-07-24 Thread Anton Volkov
ter (spi-tegra114.c: line 1125) or a write to tspi->def_command1_reg (spi-tegra114.c: line 1121)? Thank you for your time. -- Anton Volkov Linux Verification Center, ISPRAS web: http://linuxtesting.org e-mail: avol...@ispras.ru <mailto:avol...@ispras.ru>