modffi: dlsym() doesn't set errno, so use ENOENT for OSError.

pfalcon · pfalcon · commit 696eee947539 · 2015-08-31T00:20:08.000+03:00
This may be a bit confusing, as ENOENT is often rendered as "No such
file or directory", but any other code would be only more confusing.
diff --git a/unix/modffi.c b/unix/modffi.c
@@ -214,7 +214,7 @@ STATIC mp_obj_t ffimod_func(mp_uint_t n_args, const mp_obj_t *args) {
 
     void *sym = dlsym(self->handle, symname);
     if (sym == NULL) {
-        nlr_raise(mp_obj_new_exception_arg1(&mp_type_OSError, MP_OBJ_NEW_SMALL_INT(errno)));
+        nlr_raise(mp_obj_new_exception_arg1(&mp_type_OSError, MP_OBJ_NEW_SMALL_INT(ENOENT)));
     }
     return make_func(args[1], sym, args[3]);
 }
@@ -277,7 +277,7 @@ STATIC mp_obj_t ffimod_var(mp_obj_t self_in, mp_obj_t vartype_in, mp_obj_t symna
 
     void *sym = dlsym(self->handle, symname);
     if (sym == NULL) {
-        nlr_raise(mp_obj_new_exception_arg1(&mp_type_OSError, MP_OBJ_NEW_SMALL_INT(errno)));
+        nlr_raise(mp_obj_new_exception_arg1(&mp_type_OSError, MP_OBJ_NEW_SMALL_INT(ENOENT)));
     }
     mp_obj_ffivar_t *o = m_new_obj(mp_obj_ffivar_t);
     o->base.type = &ffivar_type;
@@ -294,7 +294,7 @@ STATIC mp_obj_t ffimod_addr(mp_obj_t self_in, mp_obj_t symname_in) {
 
     void *sym = dlsym(self->handle, symname);
     if (sym == NULL) {
-        nlr_raise(mp_obj_new_exception_arg1(&mp_type_OSError, MP_OBJ_NEW_SMALL_INT(errno)));
+        nlr_raise(mp_obj_new_exception_arg1(&mp_type_OSError, MP_OBJ_NEW_SMALL_INT(ENOENT)));
     }
     return mp_obj_new_int((mp_int_t)sym);
 }

Original file line number	Diff line number	Diff line change
`@@ -214,7 +214,7 @@ STATIC mp_obj_t ffimod_func(mp_uint_t n_args, const mp_obj_t *args) {`
`214`	`214`
`215`	`215`	`void *sym = dlsym(self->handle, symname);`
`216`	`216`	`if (sym == NULL) {`
`217`		`- nlr_raise(mp_obj_new_exception_arg1(&mp_type_OSError, MP_OBJ_NEW_SMALL_INT(errno)));`
	`217`	`+ nlr_raise(mp_obj_new_exception_arg1(&mp_type_OSError, MP_OBJ_NEW_SMALL_INT(ENOENT)));`
`218`	`218`	`}`
`219`	`219`	`return make_func(args[1], sym, args[3]);`
`220`	`220`	`}`
`@@ -277,7 +277,7 @@ STATIC mp_obj_t ffimod_var(mp_obj_t self_in, mp_obj_t vartype_in, mp_obj_t symna`
`277`	`277`
`278`	`278`	`void *sym = dlsym(self->handle, symname);`
`279`	`279`	`if (sym == NULL) {`
`280`		`- nlr_raise(mp_obj_new_exception_arg1(&mp_type_OSError, MP_OBJ_NEW_SMALL_INT(errno)));`
	`280`	`+ nlr_raise(mp_obj_new_exception_arg1(&mp_type_OSError, MP_OBJ_NEW_SMALL_INT(ENOENT)));`
`281`	`281`	`}`
`282`	`282`	`mp_obj_ffivar_t *o = m_new_obj(mp_obj_ffivar_t);`
`283`	`283`	`o->base.type = &ffivar_type;`
`@@ -294,7 +294,7 @@ STATIC mp_obj_t ffimod_addr(mp_obj_t self_in, mp_obj_t symname_in) {`
`294`	`294`
`295`	`295`	`void *sym = dlsym(self->handle, symname);`
`296`	`296`	`if (sym == NULL) {`
`297`		`- nlr_raise(mp_obj_new_exception_arg1(&mp_type_OSError, MP_OBJ_NEW_SMALL_INT(errno)));`
	`297`	`+ nlr_raise(mp_obj_new_exception_arg1(&mp_type_OSError, MP_OBJ_NEW_SMALL_INT(ENOENT)));`
`298`	`298`	`}`
`299`	`299`	`return mp_obj_new_int((mp_int_t)sym);`
`300`	`300`	`}`