[ovs-dev] [PATCH] ofproto: Fix use of uninitialized variable.
Ethan Jackson
ethan at nicira.com
Wed Nov 10 23:22:48 UTC 2010
Looks good.
On Wed, Nov 10, 2010 at 3:12 PM, Ben Pfaff <blp at nicira.com> wrote:
> My compiler didn't report this but Ethan's did.
>
> Reported-by: Ethan Jackson <ethan at nicira.com>
> ---
> ofproto/ofproto.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
> index e11b15d..e710c3d 100644
> --- a/ofproto/ofproto.c
> +++ b/ofproto/ofproto.c
> @@ -3676,7 +3676,7 @@ handle_vendor_stats_request(struct ofconn *ofconn,
> return ofp_mkerr(OFPET_BAD_REQUEST, OFPBRC_BAD_VENDOR);
> }
>
> - if (ntohs(nsm->header.length) < sizeof(struct nicira_stats_msg)) {
> + if (ntohs(osr->header.length) < sizeof(struct nicira_stats_msg)) {
> VLOG_WARN_RL(&rl, "truncated Nicira stats request");
> return ofp_mkerr(OFPET_BAD_REQUEST, OFPBRC_BAD_LEN);
> }
> --
> 1.7.1
>
>
More information about the dev
mailing list