bugGNU Octave - Bugs: bug #51384, strncmp is case insensitive

 
 

bug #51384: strncmp is case insensitive

Submitter:  Mike Miller <mtmiller>
Submitted:  Tue 04 Jul 2017 09:22:09 PM UTC
   
 
Category:  Octave Function Severity:  4 - Important
Priority:  5 - Normal Item Group:  Regression
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * 4.2.1 Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 05 Jul 2017 10:07:35 PM UTC, comment #3: 

Works for me, thanks for adding the tests to that, marked as fixed.

Mike Miller <mtmiller>
Group Member
Wed 05 Jul 2017 08:58:04 PM UTC, comment #2: 

I pushed the following changeset on stable, merged with default:

http://hg.savannah.gnu.org/hgweb/octave/rev/0056bcaeb01c

John W. Eaton <jwe>
Group administrator
Wed 05 Jul 2017 05:56:15 PM UTC, comment #1: 

+1 Just came here to report the same. Glad to see it's been spotted already.

Tasos Papastylianou <tpapastylianou>
Tue 04 Jul 2017 09:22:09 PM UTC, original submission:  

Somewhere between 4.0 and 4.2, the strncmp function became case insensitive.

Reported on the Debian BTS along with the attached patch provided by the reporter (https://bugs.debian.org/867212).

Fix provided in this patch looks pretty easy (liboctave/util/oct-string.cc), but needs a test case added somewhere as well.

Should be committed on the stable branch. I am happy to care of this but not today.

Mike Miller <mtmiller>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #41120:  strncmp.diff added by mtmiller (622B - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by jwe (Posted a comment)
  • -email is unavailable- added by tpapastylianou (Posted a comment)
  • -email is unavailable- added by mtmiller (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-07-05 mtmiller StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2017-07-05 jwe StatusPatch Submitted Ready For Test
    2017-07-04 mtmiller Attached File- Added strncmp.diff, #41120

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code