search

dthain / basekernel

A simple OS kernel for research, teaching, and fun.
GNU General Public License v2.0
805 stars 109 forks source link

Dup args #244

Closed jmazanec15 closed 5 years ago

jmazanec15 commented 5 years ago

Addresses #243 . Changed the order of arguments in the system call sys_object_dup to match the Unix standard. Now, the second argument dups into the first.

Only 2 user programs used this system call: pipetest and printer.

dthain commented 5 years ago

Ach, my mistake here: the arguments were in fact the right way around in the first place! http://man7.org/linux/man-pages/man2/dup.2.html

I'm sorry, I should have checked that before...

jmazanec15 commented 5 years ago

Oh ok yes I should have checked too.