Project

General

Profile

Actions

Bug #10885

closed

loader: ficl pfopendir() should push FICL_FALSE in case of error

Added by Toomas Soome over 3 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
bootloader
Start date:
2019-04-29
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:

Description

The pfopendir() is supposed to leave in stack ( fd FICL_TRUE ) if the opendir was successful and ( FICL_FALSE ) if not. The bug is that we get FICL_TRUE in both cases.

With patch we get:

ok s" boot" fopendir 
ok .s
[data stack has 2 entries, top at 0x00000000091412a0]
[0x00000000091412a0   0]:                   -1 (0xffffffffffffffff)
[0x0000000009141298   1]:                    5 (0x0000000000000005)
[data stack base at 0x0000000009141298]
ok s" boot_" fopendir 
ok .s
[data stack has 3 entries, top at 0x00000000091412a8]
[0x00000000091412a8   0]:                    0 (0x0000000000000000)
[0x00000000091412a0   1]:                   -1 (0xffffffffffffffff)
[0x0000000009141298   2]:                    5 (0x0000000000000005)
[data stack base at 0x0000000009141298]
ok 

Actions #1

Updated by Electric Monk over 3 years ago

  • Status changed from In Progress to Closed
  • % Done changed from 90 to 100

git commit c0bb4f7308f9d7f3f31eb636c73ef07c6d19b5f3

commit  c0bb4f7308f9d7f3f31eb636c73ef07c6d19b5f3
Author: Toomas Soome <tsoome@me.com>
Date:   2019-04-29T15:19:42.000Z

    10606 ficl: smatch errors: inconsistent indenting and unchecked function return
    10699 loader: smatch cleanups for ficl
    10885 loader: ficl pfopendir() should push FICL_FALSE in case of error
    Reviewed by: John Levon <john.levon@joyent.com>
    Approved by: Dan McDonald <danmcd@joyent.com>

Actions

Also available in: Atom PDF