diff options
| author | Ben Pfaff <blp@nicira.com> | 2013-01-24 13:57:37 -0800 |
|---|---|---|
| committer | Ben Pfaff <blp@nicira.com> | 2013-02-01 14:26:54 -0800 |
| commit | 0d7bb1b4f4982637b316e51a258e8efaabb3e431 (patch) | |
| tree | e502d85b50efecfbc950b5ae6381e78c51e0f9a3 /vswitchd/bridge.c | |
| parent | a6b112a8b31fdf56a73cb539c0ab41672fc71712 (diff) | |
| download | openvswitch-0d7bb1b4f4982637b316e51a258e8efaabb3e431.tar.gz | |
bridge: Rename iface_create() variable to avoid hiding parameter.
This function has a parameter 'ofp_port' and a local variable 'ofp_port',
so rename the local variable to reduce confusion.
Found by Coverity.
Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Ethan Jackson <ethan@nicira.com>
Diffstat (limited to 'vswitchd/bridge.c')
| -rw-r--r-- | vswitchd/bridge.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c index dd3099f84..607ae5e18 100644 --- a/vswitchd/bridge.c +++ b/vswitchd/bridge.c @@ -1446,9 +1446,9 @@ iface_create(struct bridge *br, struct if_cfg *if_cfg, int ofp_port) error = netdev_open(port->name, "internal", &netdev); if (!error) { - uint16_t ofp_port = if_cfg->ofport; + uint16_t fake_ofp_port = if_cfg->ofport; - ofproto_port_add(br->ofproto, netdev, &ofp_port); + ofproto_port_add(br->ofproto, netdev, &fake_ofp_port); netdev_close(netdev); } else { VLOG_WARN("could not open network device %s (%s)", |