Ticket #1543 (assigned bug)

Opened 11 months ago

Last modified 7 months ago

Check libroot for Correct Use of errno (easy)

Reported by: bonefish Owned by: kaoutsis
Priority: normal Milestone: R1
Component: System/libroot.so Version: R1 development
Cc: karma Blocked By:
Platform: All Blocking:

Description

While BeOS API functions usually return error codes directly, POSIX functions just indicate an error by a special return value (often -1 or a NULL pointer) and set errno to the actual error code. Our libroot code (src/system/libroot, particularly the posix subdirectory) needs to be reviewed in this respect.

Change History

  Changed 11 months ago by karma

  • cc karma added

  Changed 11 months ago by kaoutsis

  • owner changed from axeld to kaoutsis
  • status changed from new to assigned

I will give it a try!

  Changed 8 months ago by axeld

  • milestone changed from R1/alpha to R1

follow-up: ↓ 5   Changed 7 months ago by korli

Could you provide a status on this one please ?

in reply to: ↑ 4 ; follow-up: ↓ 6   Changed 7 months ago by kaoutsis

Replying to korli:

Could you provide a status on this one please ?

Certainly; * several additions to posix functions have been made regarding setting errno to the past 3 months (not only by me); (i don't remember exactly the particular functions names) * i have scan the posix subdirectory) for errno twice; but if it isn't urgent, i would like to check again. I think we should leave this ticket open a little longer.

in reply to: ↑ 5   Changed 7 months ago by kaoutsis

  Changed 7 months ago by korli

Ok. Full coverage is thus in progress. Thanks.

Note: See TracTickets for help on using tickets.