diff options
| author | Dan Carpenter <[email protected]> | 2023-05-03 15:11:35 +0000 |
|---|---|---|
| committer | Juergen Gross <[email protected]> | 2023-05-24 15:25:43 +0000 |
| commit | 8fafac202d18230bb9926bda48e563fd2cce2a4f (patch) | |
| tree | 7260271387adf7aab269fe3c833b889945ae57e5 /lib/string_helpers.c | |
| parent | xen/blkfront: Only check REQ_FUA for writes (diff) | |
| download | kernel-8fafac202d18230bb9926bda48e563fd2cce2a4f.tar.gz kernel-8fafac202d18230bb9926bda48e563fd2cce2a4f.zip | |
xen/pvcalls-back: fix double frees with pvcalls_new_active_socket()
In the pvcalls_new_active_socket() function, most error paths call
pvcalls_back_release_active(fedata->dev, fedata, map) which calls
sock_release() on "sock". The bug is that the caller also frees sock.
Fix this by making every error path in pvcalls_new_active_socket()
release the sock, and don't free it in the caller.
Fixes: 5db4d286a8ef ("xen/pvcalls: implement connect command")
Signed-off-by: Dan Carpenter <[email protected]>
Reviewed-by: Juergen Gross <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Juergen Gross <[email protected]>
Diffstat (limited to 'lib/string_helpers.c')
0 files changed, 0 insertions, 0 deletions