diff options
| author |   Jiri Pirko <jpirko@redhat.com> | 2011-11-17 06:32:37 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2011-11-18 14:55:03 -0500 |
| commit | f8a15af093b19b86d56933c8757cee298d0f32a8 (patch) | |
| tree | a6e8cc92edac10c26b6336671aa4d24fc4bb9e9d /drivers/net | |
| parent | team: avoid using variable-length array (diff) | |
| download | linux-dev-f8a15af093b19b86d56933c8757cee298d0f32a8.tar.xz linux-dev-f8a15af093b19b86d56933c8757cee298d0f32a8.zip | |
team: replace kmalloc+memcpy by kmemdup
Signed-off-by: Jiri Pirko <jpirko@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net')
| -rw-r--r-- | drivers/net/team/team.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/net/team/team.c b/drivers/net/team/team.c index c48ef19cac13..064155d56bce 100644 --- a/drivers/net/team/team.c +++ b/drivers/net/team/team.c @@ -104,19 +104,15 @@ int team_options_register(struct team *team, if (!dst_opts) return -ENOMEM; for (i = 0; i < option_count; i++, option++) { - struct team_option *dst_opt; - if (__team_find_option(team, option->name)) { err = -EEXIST; goto rollback; } - dst_opt = kmalloc(sizeof(*option), GFP_KERNEL); - if (!dst_opt) { + dst_opts[i] = kmemdup(option, sizeof(*option), GFP_KERNEL); + if (!dst_opts[i]) { err = -ENOMEM; goto rollback; } - memcpy(dst_opt, option, sizeof(*option)); - dst_opts[i] = dst_opt; } for (i = 0; i < option_count; i++) |