Function open() sets errno


 
Thread Tools Search this Thread
Top Forums Programming Function open() sets errno
# 1  
Old 06-27-2013
Function open() sets errno

I am opening a text file using open() system call in O_RDONLY mode.
open() returns me a valid handler but also sets errno to 13 i.e. EACCES(Permission denied).

Question is when open() is returning a valid handler then why does it sets the errno?
Should not errno be set only in case of error and open() failing?

Actually the file I am trying to open is owned by some other user.
In such a case should not open() return -1 instead of returning a valid handler?

Last edited by rupeshkp728; 06-27-2013 at 04:04 PM.. Reason: more details added
# 2  
Old 06-27-2013
Well open() returns an integer, which if -1 means errno is set to tell you what you did wrong. It sounds like your access in not good. You can set errno to zero before and test afterward. The nature of errno is that it is only set for errors, not reset on success. I usually:
Code:
if ( 0 > ( fd = open( file_name_ptr, O_RDONLY ))){
    perror( file_name_ptr );
    exit( 1 );
  }

This User Gave Thanks to DGPickett For This Post:
# 3  
Old 06-27-2013
From man 3 errno:

Code:
       The  <errno.h> header file defines the integer variable errno, which is
       set by system calls and some library functions in the event of an error
       to  indicate  what  went wrong.  Its value is significant only when the
       return value of the call indicated an error (i.e., -1 from most  system
       calls;  -1  or  NULL from most library functions); a function that suc-
       ceeds is allowed to change errno.

So, errno should never be used to check whether an error happened -- only which error. It's easy to picture more elaborate library calls changing the value of errno many times before they return... You must be exact about when and why you use it for what to get something meaningful.

I'm not sure why a successful system call would be changing errno, but it's allowed to. Perhaps it was a simplification -- "these first 4 cases will all return EACCESS, so set it first, and return immediately if any of them fail". And they never bother to change the error to success when it succeeds.

Another invalid way to use errno is checking the value of errno too late, after they've made another system call. This can give you the strange result 'ERROR: Success'.

Last edited by Corona688; 06-27-2013 at 06:48 PM..
This User Gave Thanks to Corona688 For This Post:
# 4  
Old 06-27-2013
The OP should post the operating system they're using and the code for the shortest program which reproduces the behavior (with a printf or two and the output they generate).

For all we know, the determination of "valid handler" is incorrect.

Regards,
Alister
This User Gave Thanks to alister For This Post:
# 5  
Old 06-27-2013
If you got a positive number from open(), then the errno was set earlier by some other call, unless your code does something interesting or untoward.... We do need to see some code please.

Note that system calls do not reset errno to zero, so if you do not reliably check returns codes, then it may not be possible to see where the error originated. system calls set errno ONLY when they have an error condition.
This User Gave Thanks to jim mcnamara For This Post:
# 6  
Old 06-28-2013
Thanks all for the reply.
I checked and saw errno was set by some other function called earlier.
So I understand that we must not directly check only the errno after a system call.
Instead we must accompany the check with the return value of the function.
For instance:
Code:
fd = open(...)
if(fd = -1 &&  errno == ...)
{
  ....
}


Last edited by rupeshkp728; 06-28-2013 at 11:31 AM.. Reason: details added
# 7  
Old 06-29-2013
Quote:
Originally Posted by rupeshkp728
...we must accompany the check with the return value of the function.
For instance:
Code:
fd = open(...)
if(fd = -1 &&  errno == ...)
{
  ....
}

Kind of, but you seem to just be checking for a specific error. Generally for *NIX system calls:
  • The return value tells you whether there was an error or not, and
  • errno tells you the specific type of error.

Also, be aware that you wrote "fd = -1" not "fd == -1", which is always false - not sure if this is a typo, but make sure you understand why this is an error. I always like to write the constant on the left-hand side of the comparison to try and avoid this sort of thing.

Summing up:

Code:
int fd = open(/* ... */);
if (-1 == fd) {
  // There was an error.
  switch (errno) {
  case EACCESS:
    // Access was denied.
    break;
  // ...etc...
  default:
    // Some unknown/other error.
  }
} else {
  // File opened OK.
}

This User Gave Thanks to JohnGraham For This Post:
Login or Register to Ask a Question

Previous Thread | Next Thread

9 More Discussions You Might Find Interesting

1. HP-UX

Failed to open tape device /dev/rmt/0mn:Device busy (errno = 16)

Hi, Unable to make tape backup, please help. /opt/ignite/bin/make_tape_recovery -a /dev/rmt/?mn -I -v -m tar -x inc_entire=vg00 * Creating local directories for configuration files and archive. ======= 04/25/16 16:28:08 IST Started /opt/ignite/bin/make_tape_recovery. (Mon... (4 Replies)
Discussion started by: anuragr
4 Replies

2. Shell Programming and Scripting

Open file function

Hello all, just a quick little part of code i'm writing to check if the file i'm writing too in my automatic process is not being written too manually. #!/bin/bash FUSER=$(/sbin/fuser -s /toto.tmp >/dev/null 2>&1) LSOF=$(/usr/sbin/lsof | grep -q "toto.tmp") PGREP=$(pgrep -f "toto.tmp" >... (6 Replies)
Discussion started by: maverick72
6 Replies

3. Shell Programming and Scripting

Help !! perl open function

Help Please perl Gurus, I am trying to add ungrouped passengers in a group and I creating a script however it fails on first step only I tried all the options it returns following error. syntax error at junki line 4, near "open " Execution of junki aborted due to compilation errors. ... (2 Replies)
Discussion started by: dynamax
2 Replies

4. Programming

Open function of sys/stat.h

If a process already has the entire file locked for read and write using newstruct.l_type = F_WRLCK; what would happen if another process would try to open it in read only mode using open(filename, O_RDONLY); ? I want to check if the file exists and I want it to work even if another process has... (4 Replies)
Discussion started by: cyler
4 Replies

5. UNIX and Linux Applications

Sybase help: Open client, bcp function

To begin: I use Linux The Problem: I need bcp functionality for scripts. Perl modules, such as Sybase:xfer, require ctlib which comes with Sybase Open Client. Talking with Sybase sales reps is an exercise in futility and hate. They know absolutely nothing about their own products and will... (0 Replies)
Discussion started by: Bubnoff
0 Replies

6. Programming

errno

Hey, Can I assume that for certain function calls, errno can never be set to a certain value. More specifically, can I assume that for if the stat function call fails, the errno can never be or "No space left on device." I am assuming that a read function cannot fail because of no space... (5 Replies)
Discussion started by: the_learner
5 Replies

7. UNIX and Linux Applications

(lmgrd) Can't open /usr/tmp/.flexlm/lmgrdl.4081, errno: 24

We are a tool vendor and one of our users is getting this error. The user is using several macrovision(FLEXLM) license enabled products including ours. (lmgrd) Can't open /usr/tmp/.flexlm/lmgrdl.4081, errno: 24 At this time, it stops checking out licenses, and will not respond to query's.... (2 Replies)
Discussion started by: return_user
2 Replies

8. Virtualization and Cloud Computing

Clouds (Partially Order Sets) - Streams (Linearly Ordered Sets) - Part 2

timbass Sat, 28 Jul 2007 10:07:53 +0000 Originally posted in Yahoo! CEP-Interest Here is my follow-up note on posets (partially ordered sets) and tosets (totally or linearly ordered sets) as background set theory for event processing, and in particular CEP and ESP. In my last note, we... (0 Replies)
Discussion started by: Linux Bot
0 Replies

9. Shell Programming and Scripting

function to test if file is open

I need to write a function that will work in sh/ksh shell that will test to see if a file has already been opened for writting by another user has anyone written something like this? (3 Replies)
Discussion started by: johnsonbryce
3 Replies
Login or Register to Ask a Question