Commit a7015994 authored by Jakub Kicinski's avatar Jakub Kicinski Committed by Kleber Sacilotto de Souza

bpf: fix references to free_bpf_prog_info() in comments

BugLink: https://bugs.launchpad.net/bugs/1791953

[ Upstream commit ab7f5bf0 ]

Comments in the verifier refer to free_bpf_prog_info() which
seems to have never existed in tree.  Replace it with
free_used_maps().
Signed-off-by: default avatarJakub Kicinski <jakub.kicinski@netronome.com>
Reviewed-by: default avatarQuentin Monnet <quentin.monnet@netronome.com>
Signed-off-by: default avatarDaniel Borkmann <daniel@iogearbox.net>
Signed-off-by: default avatarSasha Levin <alexander.levin@microsoft.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarStefan Bader <stefan.bader@canonical.com>
Signed-off-by: default avatarKleber Sacilotto de Souza <kleber.souza@canonical.com>
parent d34cc9de
...@@ -2133,7 +2133,7 @@ static int replace_map_fd_with_map_ptr(struct verifier_env *env) ...@@ -2133,7 +2133,7 @@ static int replace_map_fd_with_map_ptr(struct verifier_env *env)
/* hold the map. If the program is rejected by verifier, /* hold the map. If the program is rejected by verifier,
* the map will be released by release_maps() or it * the map will be released by release_maps() or it
* will be used by the valid program until it's unloaded * will be used by the valid program until it's unloaded
* and all maps are released in free_bpf_prog_info() * and all maps are released in free_used_maps()
*/ */
map = bpf_map_inc(map, false); map = bpf_map_inc(map, false);
if (IS_ERR(map)) { if (IS_ERR(map)) {
...@@ -2522,7 +2522,7 @@ int bpf_check(struct bpf_prog **prog, union bpf_attr *attr) ...@@ -2522,7 +2522,7 @@ int bpf_check(struct bpf_prog **prog, union bpf_attr *attr)
vfree(log_buf); vfree(log_buf);
if (!env->prog->aux->used_maps) if (!env->prog->aux->used_maps)
/* if we didn't copy map pointers into bpf_prog_info, release /* if we didn't copy map pointers into bpf_prog_info, release
* them now. Otherwise free_bpf_prog_info() will release them. * them now. Otherwise free_used_maps() will release them.
*/ */
release_maps(env); release_maps(env);
*prog = env->prog; *prog = env->prog;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment