diff options
| author |  Markus Elfring <elfring@users.sourceforge.net>
| 2023-12-29 13:18:56 +0100 |
|---|---|---|
| committer |  Anna Schumaker <Anna.Schumaker@Netapp.com>
| 2024-01-04 10:47:56 -0500 |
| commit | 597a421798033cd150586b5f2607389fa4052550 (patch) | |
| tree | 6d98bff5981c8e92163ae00382fc3f3ff533426c /fs/nfs | |
| parent | nfs: Remove writepage (diff) | |
| download | linux-597a421798033cd150586b5f2607389fa4052550.tar.gz linux-597a421798033cd150586b5f2607389fa4052550.tar.bz2 linux-597a421798033cd150586b5f2607389fa4052550.zip | |
rpc_pipefs: Replace one label in bl_resolve_deviceid()
The kfree() function was called in one case by
the bl_resolve_deviceid() function during error handling
even if the passed data structure member contained a null pointer.
This issue was detected by using the Coccinelle software.
Thus use an other label.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Reviewed-by: Benjamin Coddington <bcodding@redhat.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
Diffstat (limited to 'fs/nfs')
| -rw-r--r-- | fs/nfs/blocklayout/rpc_pipefs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/nfs/blocklayout/rpc_pipefs.c b/fs/nfs/blocklayout/rpc_pipefs.c index 6c977288cc28..d8d50a88de04 100644 --- a/fs/nfs/blocklayout/rpc_pipefs.c +++ b/fs/nfs/blocklayout/rpc_pipefs.c @@ -75,7 +75,7 @@ bl_resolve_deviceid(struct nfs_server *server, struct pnfs_block_volume *b, msg->len = sizeof(*bl_msg) + b->simple.len; msg->data = kzalloc(msg->len, gfp_mask); if (!msg->data) - goto out_free_data; + goto out_unlock; bl_msg = msg->data; bl_msg->type = BL_DEVICE_MOUNT; |