In the blkif_release function the bdget_disk() call might returns
a NULL ptr which might be dereferenced on bdev->bd_openers checking
Signed-off-by: Felipe Pena
---
drivers/block/xen-blkfront.c |4
1 file changed, 4 insertions(+)
diff --git a/drivers/block/xen-blkfront.c b/driv
In the blkif_release function the bdget_disk() call might returns
a NULL ptr which might be dereferenced on bdev-bd_openers checking
Signed-off-by: Felipe Pena felipe...@gmail.com
---
drivers/block/xen-blkfront.c |4
1 file changed, 4 insertions(+)
diff --git a/drivers/block/xen
linux-kernel" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
--
Regards,
Felipe Pena
--
To unsubscribe from this list: send the line "unsubscri
majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
--
Regards,
Felipe Pena
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http
There is a typo in the struct member name on assignment when checking
rtlphy->current_chan_bw == HT_CHANNEL_WIDTH_20_40, the check uses pwrgroup_ht40
for bound limit and uses pwrgroup_ht20 when assigning instead.
Signed-off-by: Felipe Pena
---
drivers/net/wireless/rtlwifi/rtl8192se/rf.c |
On rt2800_config_channel_rf53xx function the member default_power1 is checked
for bound limit, but default_power2 is used instead.
Signed-off-by: Felipe Pena
---
drivers/net/wireless/rt2x00/rt2800lib.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/wireless
On lpfc_debugfs_initialize function the dumpHostSlim member setup happens
when 'phba->sli_rev < LPFC_SLI_REV4' is true, however when it is false NULL
has been assigned to debug_dumpHBASlim instead of debug_dumpHostSlim.
Signed-off-by: Felipe Pena
---
drivers/scsi/lpfc/lpfc_debugfs.c
On Fri, Oct 18, 2013 at 7:27 PM, Larry Finger wrote:
> On 10/18/2013 05:15 PM, Felipe Pena wrote:
>>
>> There is a typo in the struct member name on assignment when checking
>> rtlphy->current_chan_bw == HT_CHANNEL_WIDTH_20_40, the check uses
>> pwrgroup_ht
In the thermal_init function when checking for thinkpad_id.ec_model,
the 'ta2' variable is being OR'd when acpi_ec_read call succeeds,
on fail it is setting 0 to 'ta1' variable instead.
Signed-off-by: Felipe Pena
---
drivers/platform/x86/thinkpad_acpi.c |2 +-
1 file changed, 1 insertion
There is a typo in the struct member name on assignment when checking
rtlphy->current_chan_bw == HT_CHANNEL_WIDTH_20_40, the check uses pwrgroup_ht40
for bound limit and uses pwrgroup_ht20 when assigning instead.
Signed-off-by: Felipe Pena
---
drivers/net/wireless/rtlwifi/rtl8192se/rf.c |
There is a typo in the struct member name on assignment when checking
rtlphy-current_chan_bw == HT_CHANNEL_WIDTH_20_40, the check uses pwrgroup_ht40
for bound limit and uses pwrgroup_ht20 when assigning instead.
Signed-off-by: Felipe Pena felipe...@gmail.com
---
drivers/net/wireless/rtlwifi
In the thermal_init function when checking for thinkpad_id.ec_model,
the 'ta2' variable is being OR'd when acpi_ec_read call succeeds,
on fail it is setting 0 to 'ta1' variable instead.
Signed-off-by: Felipe Pena felipe...@gmail.com
---
drivers/platform/x86/thinkpad_acpi.c |2 +-
1 file
On Fri, Oct 18, 2013 at 7:27 PM, Larry Finger larry.fin...@lwfinger.net wrote:
On 10/18/2013 05:15 PM, Felipe Pena wrote:
There is a typo in the struct member name on assignment when checking
rtlphy-current_chan_bw == HT_CHANNEL_WIDTH_20_40, the check uses
pwrgroup_ht40
for bound limit
On lpfc_debugfs_initialize function the dumpHostSlim member setup happens
when 'phba-sli_rev LPFC_SLI_REV4' is true, however when it is false NULL
has been assigned to debug_dumpHBASlim instead of debug_dumpHostSlim.
Signed-off-by: Felipe Pena felipe...@gmail.com
---
drivers/scsi/lpfc
On rt2800_config_channel_rf53xx function the member default_power1 is checked
for bound limit, but default_power2 is used instead.
Signed-off-by: Felipe Pena felipe...@gmail.com
---
drivers/net/wireless/rt2x00/rt2800lib.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
There is a typo in the struct member name on assignment when checking
rtlphy-current_chan_bw == HT_CHANNEL_WIDTH_20_40, the check uses pwrgroup_ht40
for bound limit and uses pwrgroup_ht20 when assigning instead.
Signed-off-by: Felipe Pena felipe...@gmail.com
---
drivers/net/wireless/rtlwifi
On the ternary expression the 'e' variable could be NULL dereferenced,
when b43_nphy_get_rf_ctl_over_rev7 function returns NULL.
Signed-off-by: Felipe Pena
---
drivers/net/wireless/b43/phy_n.c |3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/net/wireless/b43
On the ternary expression the 'e' variable could be NULL dereferenced,
when b43_nphy_get_rf_ctl_over_rev7 function returns NULL.
Signed-off-by: Felipe Pena felipe...@gmail.com
---
drivers/net/wireless/b43/phy_n.c |3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers
In the lpfc_ct_free_iocb function after freeing associated memory to the
ctiocb->context3, the ctiocb->context1 is set to NULL instead of context3.
Signed-off-by: Felipe Pena
---
drivers/scsi/lpfc/lpfc_ct.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scs
There is a typo where the checking for priv->ReAssociationRequestRetryCnt must
be, it was checking for priv->AssociationRequestRetryCnt instead.
Signed-off-by: Felipe Pena
---
drivers/net/wireless/atmel.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drive
There is a wrong check on can_attach_cache() function which
bucket_size is compared to block_size value.
Signed-off-by: Felipe Pena
---
drivers/md/bcache/super.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/md/bcache/super.c b/drivers/md/bcache/super.c
index
The check for calling free_page() on hv_context.synic_event_page[cpu] is the
same for hv_context.synic_message_page[cpu], like a copy-paste error.
Signed-off-by: Felipe Pena
---
drivers/hv/hv.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/hv/hv.c b/drivers/hv
There is a typo where the checking for priv-ReAssociationRequestRetryCnt must
be, it was checking for priv-AssociationRequestRetryCnt instead.
Signed-off-by: Felipe Pena felipe...@gmail.com
---
drivers/net/wireless/atmel.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
In the lpfc_ct_free_iocb function after freeing associated memory to the
ctiocb-context3, the ctiocb-context1 is set to NULL instead of context3.
Signed-off-by: Felipe Pena felipe...@gmail.com
---
drivers/scsi/lpfc/lpfc_ct.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
The check for calling free_page() on hv_context.synic_event_page[cpu] is the
same for hv_context.synic_message_page[cpu], like a copy-paste error.
Signed-off-by: Felipe Pena felipe...@gmail.com
---
drivers/hv/hv.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/hv
There is a wrong check on can_attach_cache() function which
bucket_size is compared to block_size value.
Signed-off-by: Felipe Pena felipe...@gmail.com
---
drivers/md/bcache/super.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/md/bcache/super.c b/drivers/md
Commit-ID: 1df9297c8535a5bb2b776381e63d8334f87d4abe
Gitweb: http://git.kernel.org/tip/1df9297c8535a5bb2b776381e63d8334f87d4abe
Author: Felipe Pena
AuthorDate: Wed, 9 Oct 2013 23:00:38 -0300
Committer: Arnaldo Carvalho de Melo
CommitDate: Mon, 14 Oct 2013 10:28:54 -0300
perf tests: Fix
The variable txlen was used instead of rxlen in a bound checking. (copy-paste
error)
Signed-off-by: Felipe Pena
---
drivers/media/usb/dvb-usb/technisat-usb2.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/usb/dvb-usb/technisat-usb2.c
b/drivers/media/usb
Fix for a memory leak on tracing_data_get() function when returning NULL
explicitly
Signed-off-by: Felipe Pena
---
tools/perf/util/trace-event-info.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/tools/perf/util/trace-event-info.c
b/tools/perf/util/trace
Hi,
On Mon, Oct 14, 2013 at 8:59 AM, Mark Brown wrote:
> On Sat, Oct 12, 2013 at 07:35:06PM -0300, Felipe Pena wrote:
>> When audmux_clk is used and clk_prepare_enable function succeed,
>> the memory alloc'd to buf variable is leaked
>
> Applied, thanks. Please try
Hi,
On Mon, Oct 14, 2013 at 8:59 AM, Mark Brown broo...@kernel.org wrote:
On Sat, Oct 12, 2013 at 07:35:06PM -0300, Felipe Pena wrote:
When audmux_clk is used and clk_prepare_enable function succeed,
the memory alloc'd to buf variable is leaked
Applied, thanks. Please try to use subject
Fix for a memory leak on tracing_data_get() function when returning NULL
explicitly
Signed-off-by: Felipe Pena felipe...@gmail.com
---
tools/perf/util/trace-event-info.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/tools/perf/util/trace-event-info.c
b/tools
The variable txlen was used instead of rxlen in a bound checking. (copy-paste
error)
Signed-off-by: Felipe Pena felipe...@gmail.com
---
drivers/media/usb/dvb-usb/technisat-usb2.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/usb/dvb-usb/technisat-usb2.c
b
Commit-ID: 1df9297c8535a5bb2b776381e63d8334f87d4abe
Gitweb: http://git.kernel.org/tip/1df9297c8535a5bb2b776381e63d8334f87d4abe
Author: Felipe Pena felipe...@gmail.com
AuthorDate: Wed, 9 Oct 2013 23:00:38 -0300
Committer: Arnaldo Carvalho de Melo a...@redhat.com
CommitDate: Mon, 14 Oct
The tx_desc variable is being used to access its type member
after a kmem_cache_free call
Signed-off-by: Felipe Pena
---
drivers/infiniband/ulp/iser/iser_initiator.c |8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/drivers/infiniband/ulp/iser/iser_initiator.c
b
The tx_desc variable is being used to access its type member
after a kmem_cache_free call
Signed-off-by: Felipe Pena felipe...@gmail.com
---
drivers/infiniband/ulp/iser/iser_initiator.c |8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/drivers/infiniband/ulp/iser
When audmux_clk is used and clk_prepare_enable function succeed,
the memory alloc'd to buf variable is leaked
Signed-off-by: Felipe Pena
---
sound/soc/fsl/imx-audmux.c |9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/sound/soc/fsl/imx-audmux.c b/sound/soc/fsl/imx
When audmux_clk is used and clk_prepare_enable function succeed,
the memory alloc'd to buf variable is leaked
Signed-off-by: Felipe Pena
---
sound/soc/fsl/imx-audmux.c |9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/sound/soc/fsl/imx-audmux.c b/sound/soc/fsl/imx
When audmux_clk is used and clk_prepare_enable function succeed,
the memory alloc'd to buf variable is leaked
Signed-off-by: Felipe Pena felipe...@gmail.com
---
sound/soc/fsl/imx-audmux.c |9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/sound/soc/fsl/imx-audmux.c b
When audmux_clk is used and clk_prepare_enable function succeed,
the memory alloc'd to buf variable is leaked
Signed-off-by: Felipe Pena felipe...@gmail.com
---
sound/soc/fsl/imx-audmux.c |9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/sound/soc/fsl/imx-audmux.c b
The err variable is intended to receive the timer_create() return before
checking it
Signed-off-by: Felipe Pena
---
tools/testing/selftests/timers/posix_timers.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/testing/selftests/timers/posix_timers.c
b/tools/testing
Fix for a memory leak on tracing_data_get() function when returning NULL
explicitly
Signed-off-by: Felipe Pena
---
tools/perf/util/trace-event-info.c |7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/tools/perf/util/trace-event-info.c
b/tools/perf/util/trace-event
Fix for a memory leak on tracing_data_get() function when returning NULL
explicitly
Signed-off-by: Felipe Pena felipe...@gmail.com
---
tools/perf/util/trace-event-info.c |7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/tools/perf/util/trace-event-info.c
b/tools
The err variable is intended to receive the timer_create() return before
checking it
Signed-off-by: Felipe Pena felipe...@gmail.com
---
tools/testing/selftests/timers/posix_timers.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/testing/selftests/timers
Fix for a memory leak on tracing_data_get() function when returning NULL
explicitly
Signed-off-by: Felipe Pena
---
tools/perf/util/trace-event-info.c |8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/tools/perf/util/trace-event-info.c
b/tools/perf/util/trace-event
Fix for a memory leak on test_file() function in dso-data.c
Signed-off-by: Felipe Pena
---
tools/perf/tests/dso-data.c |1 +
1 file changed, 1 insertion(+)
diff --git a/tools/perf/tests/dso-data.c b/tools/perf/tests/dso-data.c
index dffe055..9cc81a3 100644
--- a/tools/perf/tests/dso-data.c
e-mail para kernel-br+unsubscr...@googlegroups.com.
> Para postar neste grupo, envie um e-mail para kernel...@googlegroups.com.
> Para ver esta discussão na web, acesse
> https://groups.google.com/d/msgid/kernel-br/1381360387-27535-1-git-send-email-geyslan%40gmail.com.
> Para obter m
-27535-1-git-send-email-geyslan%40gmail.com.
Para obter mais opções, acesse https://groups.google.com/groups/opt_out.
--
Regards,
Felipe Pena
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http
Fix for a memory leak on test_file() function in dso-data.c
Signed-off-by: Felipe Pena felipe...@gmail.com
---
tools/perf/tests/dso-data.c |1 +
1 file changed, 1 insertion(+)
diff --git a/tools/perf/tests/dso-data.c b/tools/perf/tests/dso-data.c
index dffe055..9cc81a3 100644
--- a/tools
Fix for a memory leak on tracing_data_get() function when returning NULL
explicitly
Signed-off-by: Felipe Pena felipe...@gmail.com
---
tools/perf/util/trace-event-info.c |8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/tools/perf/util/trace-event-info.c
b/tools
The zynq_clk_register_fclk function can leak memory (fclk_lock) when unable
to alloc memory for fclk_gate_lock
Signed-off-by: Felipe Pena
Acked-by: Sören Brinkmann
---
drivers/clk/zynq/clkc.c | 16 +++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/drivers/clk
>> + }
>> return >ttm.ttm;
>> }
>> --
>> 1.8.4
>>
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http:/
majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
--
Regards,
Felipe Pena
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http
The zynq_clk_register_fclk function can leak memory (fclk_lock) when unable
to alloc memory for fclk_gate_lock
Signed-off-by: Felipe Pena felipe...@gmail.com
Acked-by: Sören Brinkmann soren.brinkm...@xilinx.com
---
drivers/clk/zynq/clkc.c | 16 +++-
1 file changed, 15 insertions
The zynq_clk_register_fclk function can leak memory (fclk_lock) when unable
to alloc memory for fclk_gate_lock
Signed-off-by: Felipe Pena
---
drivers/clk/zynq/clkc.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/clk/zynq/clkc.c b/drivers/clk/zynq/clkc.c
index cc40fe6..7ea4b5c
The zynq_clk_register_fclk function can leak memory (fclk_lock) when unable
to alloc memory for fclk_gate_lock
Signed-off-by: Felipe Pena felipe...@gmail.com
---
drivers/clk/zynq/clkc.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/clk/zynq/clkc.c b/drivers/clk/zynq/clkc.c
The FAULT_FLAG_WRITE flag has been set based on uninitialized variable
Signed-off-by: Felipe Pena
---
arch/parisc/mm/fault.c |5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/arch/parisc/mm/fault.c b/arch/parisc/mm/fault.c
index d10d27a..00c0ed3 100644
--- a/arch
Hello Johannes,
On Sun, Sep 22, 2013 at 7:58 PM, Johannes Weiner wrote:
> Hello Felipe,
>
> On Sun, Sep 22, 2013 at 03:17:46PM -0300, Felipe Pena wrote:
>> The FAULT_FLAG_WRITE flag has been set based on uninitialized variable
>
> Oops, you are right.
>
>&
The FAULT_FLAG_WRITE flag has been set based on uninitialized variable
Signed-off-by: Felipe Pena
---
arch/parisc/mm/fault.c |5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/arch/parisc/mm/fault.c b/arch/parisc/mm/fault.c
index d10d27a..6b38026 100644
--- a/arch
The FAULT_FLAG_WRITE flag has been set based on uninitialized variable
Signed-off-by: Felipe Pena felipe...@gmail.com
---
arch/parisc/mm/fault.c |5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/arch/parisc/mm/fault.c b/arch/parisc/mm/fault.c
index d10d27a..6b38026
Hello Johannes,
On Sun, Sep 22, 2013 at 7:58 PM, Johannes Weiner han...@cmpxchg.org wrote:
Hello Felipe,
On Sun, Sep 22, 2013 at 03:17:46PM -0300, Felipe Pena wrote:
The FAULT_FLAG_WRITE flag has been set based on uninitialized variable
Oops, you are right.
Signed-off-by: Felipe Pena
The FAULT_FLAG_WRITE flag has been set based on uninitialized variable
Signed-off-by: Felipe Pena felipe...@gmail.com
---
arch/parisc/mm/fault.c |5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/arch/parisc/mm/fault.c b/arch/parisc/mm/fault.c
index d10d27a..00c0ed3
62 matches
Mail list logo