Re: [PATCH] octeontx2-af: Fix a resource leak in an error handling path in 'cgx_probe()'

From: Christophe JAILLET
Date: Fri Jan 04 2019 - 17:05:54 EST


Le 04/01/2019 Ã 21:46, David Miller a ÃcritÂ:
From: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
Date: Sat, 29 Dec 2018 17:42:22 +0100

If an error occurs after the call to 'pci_alloc_irq_vectors()', we must
call 'pci_free_irq_vectors()' in order to avoid a resource leak.

The same sequence is already in place in the corresponding 'cgx_remove()'
function.

Fixes: 1463f382f58d ("octeontx2-af: Add support for CGX link management")
Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
Applied and queued up for -stable.

Please fix the date and time on your computer, putting improper dates into
the email headers of your patch postings makes your patch appear out
of order in my patchwork queue.

Thank you.

Hi,

yes, I've spotted this time issue.
For some reason my VM get out of synch when restored from a saved state.
It was not the case before. It is likely due to an update of VirtualBox (i.e. 5.x --> 6.0) performed a few days ago.
Will try to give it a look.

For now, I have to manually do some ntpdate to get the time/date fixed... until the next save/restore of the VM.

Sorry for the inconvenience.

CJ