Bug 5813 - thread-safe issues
Summary: thread-safe issues
Status: NEW
Alias: None
Product: Samba 3.5
Classification: Unclassified
Component: libsmbclient (show other bugs)
Version: unspecified
Hardware: Other Linux
: P3 normal
Target Milestone: ---
Assignee: Derrell Lipman
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks: 6358
  Show dependency treegraph
 
Reported: 2008-10-06 11:31 UTC by Derrell Lipman
Modified: 2010-04-30 11:08 UTC (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Derrell Lipman 2008-10-06 11:31:57 UTC
Once the samba core corrects thread safety issues, a few remain in libsmbclient.  From mailing list discussion with David Collier-Brown:

    Ok, then there are only a few issues left in the whole directory:


Great, thanks!
 


    froggy> cat libsmbclient.report

    atof - convert string to double-precision number
       MT-Level: MT-Safe as  long  as setlocale(3C) is not
       called to change the locale.


How in the world can atof() be non-thread-safe?  Why would locale have anything to do with it?  It returns a double and I can't think of any reason why it would need to use any non-thread-safe features...???  Is strtod() deemed to be more thread safe?

(note: apparently, some POSIX system were determined to have this failure.  TBD: do we care about such brain damage?)

 


    ctime - convert date and time to string
       MT-Level: MT-Safe with exceptions
       the ctime(), gmtime(), and localtime()  functions
       are  safe  to  use  in multithread applications because they
       employ  thread-specific  data.   However,   their   use   is
       discouraged  because  standards  do  not  require them to be
       thread-safe. Use  asctime_r(), gmtime_r() and ctime_r(),


Yes, this is a real issue if the implementation does not use thread-specific data.  The problem is that the _r functions are not standardized either (even their existence) so using them doesn't really help.  I suppose one could add ctime_r to libreplace...  

(some real systems were determined to not use thread-specific data in this function.)

    getpass - read a string of characters without echo
       MT-Level: MT-Unsafe

    Found in:
    "./clidfs.c",
         209:


I think there's a more serious issue here.  getpass() is only called if no password has been provided and kerberos is not in use.  Those things are determined, however, via members of a static structure in cli_dfs.c (cm_creds) which is clearly not appropriate for different threads providing different credentials.  This is the kind of samba core issue (not libsmbclient-specific code) I was talking about, that I think Jeremy indicated that he hopes to be working on.