From patchwork Tue Nov 25 19:34:48 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Martin Galvan X-Patchwork-Id: 3910 Received: (qmail 3583 invoked by alias); 25 Nov 2014 19:34:59 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 3574 invoked by uid 89); 25 Nov 2014 19:34:58 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=AWL, BAYES_00, RCVD_IN_DNSWL_LOW, SPF_SOFTFAIL autolearn=no version=3.3.2 X-HELO: mail-qc0-f175.google.com Received: from mail-qc0-f175.google.com (HELO mail-qc0-f175.google.com) (209.85.216.175) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Tue, 25 Nov 2014 19:34:57 +0000 Received: by mail-qc0-f175.google.com with SMTP id b13so1091019qcw.34 for ; Tue, 25 Nov 2014 11:34:55 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=l3Hy9qXO2IFv8tGuv7/Fdy8saWqWSXea00tTttGXGjw=; b=IUA4BJBN6tvdcIX3ZasWteJldNOc6VqJzKMnYGW0e4uL5SC5a5DBsC16lcfTBKmwRV Udvh82s+14cRiBpq6kSf7Xw2q04Xlz8NsadoPcUTyl5nkpeL1RO2eys1h1Pqurr6SVc+ BSlwr5gfYd3jDoXq0r96VlQc49ThunVXsiaQwrV24UIVRTpVHl0kbPztBcZvoxx0Feqd tgMyxUo9nWKa29jOC4nDUkpeiEiFBlftZW8uyjAHTqDuu9sqd0oDis7Hcl7JZaRfOLgP j4VbYTnhzMzZ7zqdFuVFIUHb2QGlIy68DYyAO5uM/Hj+jzV4G41VRiDZrm1jioqRboBF nNmQ== X-Gm-Message-State: ALoCoQkMYf3c2MxesuDA4iNUkLg/4VKVs1yy+4l769aa8rlgl5+J0H2XWWty7R38+On4F/fWQl/3 X-Received: by 10.140.42.135 with SMTP id c7mr38336023qga.7.1416944095693; Tue, 25 Nov 2014 11:34:55 -0800 (PST) Received: from martin-galvan.dominio.tallertechnologies.com ([200.69.202.173]) by mx.google.com with ESMTPSA id e45sm1511140qgd.35.2014.11.25.11.34.53 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Tue, 25 Nov 2014 11:34:54 -0800 (PST) From: Martin Galvan To: gdb-patches@sourceware.org Cc: daniel.gutson@tallertechnologies.com Subject: [PATCH] frame.c: Fix the check for FID_STACK_INVALID in frame_id_eq(). Date: Tue, 25 Nov 2014 16:34:48 -0300 Message-Id: <1416944088-8750-1-git-send-email-martin.galvan@tallertechnologies.com> I noticed in frame_id_eq() we were checking for the "l" frame_id being invalid twice instead of checking both "l" and "r", so I corrected it. gdb/ChangeLog: 2014-11-25 Martin Galvan * frame.c (frame_id_eq): Fix the check for FID_STACK_INVALID. diff --git a/gdb/frame.c b/gdb/frame.c index 0663af9..135b820 100644 --- a/gdb/frame.c +++ b/gdb/frame.c @@ -614,7 +614,7 @@ frame_id_eq (struct frame_id l, struct frame_id r) outer_frame_id. */ eq = 1; else if (l.stack_status == FID_STACK_INVALID - || l.stack_status == FID_STACK_INVALID) + || r.stack_status == FID_STACK_INVALID) /* Like a NaN, if either ID is invalid, the result is false. Note that a frame ID is invalid iff it is the null frame ID. */ eq = 0;