bugGNU Octave - Bugs: bug #56703, winqueryreg does not return...

 
 

bug #56703: winqueryreg does not return correct DWORD data

Submitter:  John Donoghue <lostbard>
Submitted:  Wed 31 Jul 2019 04:57:38 PM UTC
   
 
Category:  Octave Function Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Fixed Assigned to:  lostbard
Originator Name:  JohnD Open/Closed:  * Closed
Release:  * 5.1.0 Operating System:  * Microsoft Windows
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Post a Comment

Add a New Comment Rich Markup
   

Discussion

Tue 17 Sep 2019 08:58:20 PM UTC, comment #5: 

I merged stable to default afterwards so that the development version is also caught up.  Marking as fixed and closing report.

Rik <rik5>
Group administrator
Tue 17 Sep 2019 08:47:22 PM UTC, comment #4: 

I'd forgotten about this one :)

Pushed: http://hg.savanna ... /rev/6bbbfba55989

John Donoghue <lostbard>
Group Member
Tue 17 Sep 2019 06:43:36 PM UTC, comment #3: 

@John: Reviewed your patch.  It looks good.  Go ahead and commit it to stable.

Rik <rik5>
Group administrator
Wed 31 Jul 2019 07:46:24 PM UTC, comment #2: 

After patch applied to stable:

> winqueryreg ("HKLM", 'SOFTWARE\Microsoft\Windows NT\CurrentVersion', 'InstallDate')


ans = 1554817635


John Donoghue <lostbard>
Group Member
Wed 31 Jul 2019 05:05:22 PM UTC, comment #1: 

Not compiled yet, but patch simlar to whats in the windows package:



(file #47281)

John Donoghue <lostbard>
Group Member
Wed 31 Jul 2019 04:57:38 PM UTC, original submission:  


Return  DWORD data from the registry, where that value is more than 255:

For example:

winqueryreg ("HKLM", 'SOFTWARE\Microsoft\Windows NT\CurrentVersion', 'InstallDate')

Actual value using regedit:

1554817635


Using the function in octave 5.1:

ans = 99


using a recently updated windows package in Octave 4.2:

ans =   1.5548e+009



Looking at the code, it seems to be getting the value of the first byte of the read buffer rather than getting as a DWORD.

John Donoghue <lostbard>
Group Member

 

Attached Files

Attached Files
file #47281:  winquery.patch added by lostbard (915B - application/octet-stream)

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

Attach Files:
   
   
Comment:
   

 

Dependencies

This item does not depend on any other items.

No items depend on this one.

 

Mail Notification Carbon-Copy List

Carbon-Copy List
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by lostbard (Submitted the item)
  •  

    Votes

    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.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

    History

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-09-17 rik5 StatusPatch Reviewed Fixed
        Open/ClosedOpen Closed
    2019-09-17 rik5 StatusPatch Submitted Patch Reviewed
    2019-07-31 lostbard Attached File- Added winquery.patch, #47281
        StatusIn Progress Patch Submitted

    Back to the top

    Powered by Savane 3.16-598c.
    Corresponding source code