edaa6f95cc90f4d1279e72bfb82717fcd1732da2
[oftpd.git] / ChangeLog
1 2007-02-06  Marcus Brinkmann  <marcus@g10code.de>
2
3         * configure.in (AC_CHECK_HEADERS): Check for sys/sendfile.h.
4         * src/file_list.c (file_nlst): Remove unused variable HIDDEN.
5         * src/ftp_session.c [HAVE_SYS_SENDFILE_H]: Include
6         <sys/sendfile.h>.
7         (do_user, do_pass): Remove unused variable addr_port.
8         (do_eprt): Remove unused variable HOST_PORT.
9         (do_retr): Remove unused variable host_port.  Adjust format output
10         parameter for timespec members, cast these to the format type.
11         [!HAVE_FREEBSD_SENDFILE]: Do not define otherwise unused variable
12         AMT_SENT.
13         * src/ftp_listener.c (addr2string) [!INET6]: Do not define
14         variables ERROR and ADDR to suppress compiler warning.
15         (addr_to_string): Replace with prototype for addr2string.
16         * src/oftpd.c: Remove extra argument EXE_NAME in syslog
17         invocation.
18         (my_syslog_fd): Remove unused global variable.
19         * src/daemon_assert.c: Include <stdlib.h>.
20
21 2006-03-09  Werner Koch  <wk@g10code.de>
22
23         * configure.in: Define ERRBUF_SIZE.
24
25         * src/oftpd.c: Add option --version.
26
27 2005-04-30  Werner Koch  <wk@g10code.de>
28
29         Bumped version to 0.3.7-wk1.
30
31         * src/file_list.c (file_nlst, file_list): Fixed the paranoia test for
32         overlong directories.
33
34 2005-04-30  Werner Koch  <wk@g10code.de>
35
36         Ported patches from Shane Kerr's 0.3.7 release:
37         
38         * configure.in: Disabled support for sendfile() on FreeBSD.  It
39         appears to me that there's a bug in the sendfile() user-level
40         library.  I would welcome advice from any FreeBSD developer
41         wishing to set me straight.  
42
43         * src/ftp_session.c (do_size): Changed SIZE command to return
44         error for directories.
45         * src/file_list.c: #ifdef wrap GLOB_ABORTED for really old gcc
46         compilers (Slackware or older installations).
47         (skip_ls_options): Add code to skip command line options that are
48         intended for "ls".
49
50 2004-01-12  Werner Koch  <wk@gnupg.org>
51
52         * src/ftp_listener.c (connection_acceptor): Don't increase the
53         error counter for EMFILE conditions but sleep a few seconds.
54
55 2003-08-13  Werner Koch  <wk@gnupg.org>
56
57         * src/oftpd.c (reopen_syslog_hack): Removed.
58         (init_syslog_hack): Removed.  That was a whole stupid thing by me
59         as I didn'd realized that we are running chroot and so the
60         /dev/log socket could not be reopened by libc.
61         (main): Print a warning if there is no /dev directory below the
62         chroot.
63
64         * src/file_list.c (file_nlst,file_list): Don't list "/dev".
65         * src/ftp_session.c (change_dir, do_retr)
66         (do_size, do_mdtm): Likewise.
67
68 2003-08-12  Werner Koch  <wk@gnupg.org>
69
70         * src/oftpd.c (reopen_syslog_hack): Simply comparing the fd is not
71         enough due to race conditions.  So always check whether the syslog
72         fd is still bound to a unix domain socket.
73
74 2003-08-09  Werner Koch  <wk@gnupg.org>
75
76         * src/oftpd.c (main): Give LOG_FACILITY file scope.
77         (init_syslog_hack): New
78         (reopen_syslog_hack): New.
79         * src/ftp_session.c (set_pasv, open_connection): Insert
80         reopen_syslog_hack calls.
81         * src/ftp_listener.c (connection_acceptor, ftp_listener_init): Ditto.
82         * src/ftp_session.c (open_connection): Ditto.
83         
84         * src/file_list.c: As an extra safeguard check the provided
85         dir_len.
86         (file_nlst): Ditto.
87         (fdprintf): Extra paranoid safeguard for buggy vsnprintfs.
88
89 2003-05-22  Werner Koch  <wk@gnupg.org>
90
91         * src/ftp_command.c (parse_offset): Fixed offset checking.
92
93         * configure.in (AC_CHECK_SIZEOF): Add tests for off_t,
94         unsigned long and unsigned long long. 
95
96 2003-05-06  Werner Koch  <wk@gnupg.org>
97
98         * src/ftp_session.c (do_lprt): Do not set the port after emitting
99         an error.  Fixes an assertion failure in ip_equal.
100
101 2003-05-05  Werner Koch  <wk@gnupg.org>
102
103 - New option -r, --pasv-range to limit the range for passive ports to
104   specific firewall rules.
105
106 2001-05-28, Shane Kerr <shane@time-travellers.org>
107 - Version 0.3.6 released.
108 - Beat FreeBSD into submission.  Now compiles (and runs!) on FreeBSD
109   out of the box.
110
111 2001-05-11, Shane Kerr <shane@time-travellers.org>
112 - Changed REST to only work for IMAGE (i.e. binary) mode.
113 - Added the SIZE command.
114 - Added the MDTM (modification time) command.
115
116 2001-04-20, Shane Kerr <shane@time-travellers.org>
117 - Moved SIGPIPE ignore to run even in --nodetach mode.  Needed to avoid
118   unwanted signals on TCP disconnects.
119
120 2001-04-19, Shane Kerr <shane@time-travellers.org>
121 - Added more logging: all client messages are logged with address and
122   port, and all server responses are also logged.  This occurs at DEBUG
123   level, as it is probably not normally of interest.
124
125 2001-04-18, Shane Kerr <shane@time-travellers.org>
126 - Version 0.3.5 released.
127 - Changed resume code to reset offset to 0 after each file transfer.
128 - Set accept() socket to NONBLOCK to insure the listener doesn't get
129   stuck waiting for a connection.  This is documented in the NOTES
130   section for the accept() call in Linux.
131 - Added sendfile() support on Linux systems.
132
133 2001-04-14, Shane Kerr <shane@time-travellers.org>
134 - Fixed a bug caused by parsing EPRT commands.  The server doesn't
135   actually allow these commands, but it bravely attempts to parse them
136   anyway.  A special shout out to Anders Nordby for finding this.  The
137   error caused the server to terminate on an assert() fail, which is
138   good because no server corruption happened, but it's bad because the
139   server terminated without any indication.  Therefore....
140 - Changed most assert() calls to daemon_assert() calls.  These terminate
141   the application as before, but log to syslog and STDERR.
142 - Added ability to run as a non-daemon process.  This will allow the
143   server to be started from init, for instance.
144 - Added support for SIGTERM and SIGINT.  If one of these is received,
145   then the server closes the port 21 socket - meaning that it will
146   accept no further connections.  When all existing connections have
147   closed, then the server exits.  A new server may be started before
148   this occurs, to handle new connections.  This allows very short
149   downtimes:
150       # killall oftpd; sleep 1; /etc/init.d/oftpd start
151   This will result in a 1 second period of time where no new connections
152   are accepted, with no existing connections closed.  It's probably best
153   to wait for this second, to allow the signal time to arrive.  :)
154
155 2001-04-08, Shane Kerr <shane@time-travellers.org>
156 - Changed check for '/' in path - was improperly allowing escaped '/'
157   characters through the check!
158 - Set TCP_NODELAY flag on socket - much reduced latency on control
159   channel for high-speed connections.
160 - New and improved init script from Eric Jensen.
161
162 2001-04-07, Anders Nordby <anders@fix.no>
163 - FreeBSD port.
164
165 2001-04-03, Shane Kerr <shane@time-travellers.org>
166 - Version 0.3.4 released.
167 - Changed FTP listener code to attempt to continue processing when
168   formerly fatal errors occur.
169 - Added man page.
170 - Added Red Hat init script donated by Eric Jensen to release.
171
172 2001-03-28, Shane Kerr <shane@time-travellers.org>
173 - Version 0.3.3 released.
174 - Use IP address and port of the control connection port for default
175   Data Transfer Port, as defined by RFC 959.  Note that the default
176   server process DTP is never used in oftpd, but that's not a problem as
177   I read the RFC.
178 - Use config.h
179
180 2001-03-27, Matthew Danish <mdanish@andrew.cmu.edu>
181 - configure.in and af_portability.h fixes to allow compilation on
182   systems with both RFC 2553 and XNET formats
183
184 2001-03-27, Shane Kerr <shane@time-travellers.org>
185 - Fixed bug where wrong server address was used on initalization of
186   ftp_server class.  Passive probably didn't work in the 0.3.x series
187   for IPv4!
188
189 2001-03-26, Shane Kerr <shane@time-travellers.org>
190 - Version 0.3.2 released.
191 - Minor additions for better error reporting on error creating threads
192   in listener, as well as checking success on setgid() and setuid().  I
193   can't believe I wasn't!!!
194 - Set syslog() to use FTP_DAEMON
195
196 2001-03-23, Shane Kerr <shane@time-travellers.org>
197 - Fixed configure bug on Debian systems.  Special thanks to Matthew
198   Danish for helping me on this one.
199
200 2001-03-20, Matthew Danish <mdanish@andrew.cmu.edu>
201 - Clean up parsing of FTP address in ftp_listener.c
202 - Added options to specify interface and maximum number of client
203   connections at startup.
204 - Added --long-style command-line options.
205
206 2001-03-21, Shane Kerr <shane@time-travellers.org>
207 - The "Spring Fever" edition, version 0.3.1 released.
208
209   Added workaround for the evil glob() denial-of-service attack.  I
210   chose the simple method of preventing file listing with both path
211   separator and wildcards.  Either is okay, but not both.  I don't
212   expect this to cause problems with legitimate use, but I could be
213   wrong.  It was much simpler than my other thought.  :)
214
215   Changed code so that ECONNABORTED and ECONNRESET errors don't increase
216   the error count.  This is to prevent a malicious client from taking a
217   server down by sending a large number of connection requests that it
218   then aborts.
219
220   Integrated code from Matthew Danish <mdanish@andrew.cmu.edu> that
221   allows the user to specify the port to use from the command line.
222   Thanks!
223
224   Warnings from 0.3.0 still apply.  Use with caution.  Spring is in the
225   air, after all.
226
227 2001-03-20, Shane Kerr <shane@time-travellers.org>
228 - Version 0.3.0 released.  This is almost guaranteed not to work, as
229   I've not tested it with clients that support the new features.  But I
230   decided that after a year, I wasn't living up to the "release early,
231   release often" motto of open source.
232
233   In addition to working in the changes since the last release, I've
234   done:
235
236   o removed free list from ftp_listener; simple is better
237   o use a random port rather than incrememting sequence; useful for
238     preventing classes of data hijacking attacks
239
240   There is a potential DOS attack against all versions, see the BUGS
241   file.  This will be fixed ASAP (hopefully tomorrow).
242
243 2001-02-22, Matthew Danish <mdanish@andrew.cmu.edu>
244 - Fix for missing <sys/types.h> check in configure.in
245
246 2000-12-13, Mauro Tortonesi <mauro@ferrara.linux.it>
247 - Added IPv6 support (EPRT & EPSV - RFC2428)
248 - Added LPRT and LPSV support (RFC1639)
249
250 2000-08-25, Shane Kerr <shane@time-travellers.org>
251 - Fixed telnet module to properly handle telnet escape sequences (since
252   no clients actually use this, it wasn't really a problem)
253
254 2000-06-01, Shane Kerr <shane@time-travellers.org>
255 - Beau Kuiper, author of muddleftpd, sent me a fix for the Netscape
256   timing bug.  Thanks!!!
257
258 2000-04-03, Shane Kerr <shane@time-travellers.org>
259 - Version 0.2.0 released
260
261 2000-03-30, Shane Kerr <shane@time-travellers.org>
262 - Changed README send to only send on directory change if you actually
263   change to a different directory.  That is, "CWD ." does not send a
264   README file.  The reason for this is that many clients perform a 
265   "CWD /" as soon as it connects, which cause the README to be sent
266   twice - yuck!
267 - Added missing check for closed file descriptor in write_fully().
268 - Converted use of FILE pointers to file descriptors.  This wasn't done
269   in the most efficient possible method (i.e. with buffers), but it does
270   remove the maximum FILE limitation from the server.
271
272 2000-03-29, Shane Kerr <shane@time-travellers.org>
273 - Added -D_REENTRANT flag to compile options.
274 - Added an error data type to return details about errors that occur 
275   in module initialization.
276 - Changed telnet module to never drop characters on outbound, even if
277   we have a ton of DO and WILL commands from the other end.
278
279 2000-03-27, Shane Kerr <shane@time-travellers.org>
280 - Fixed bug in telnet code when sending "" string.
281 - Now sends README to client if file is in directory on connect
282   or directory change.
283 - Simplified watchdog a bit by adding pointer to watchdog in the
284   watched structure.
285 - Wrapped invariant() methods with #ifndef NDEBUG so they won't
286   get compiled in non-debug code (in case there's ever a 
287   non-debug version)
288
289 2000-03-22, Shane Kerr <shane@time-travellers.org>
290 - Changed NLST and LIST to silently drop a file specification that
291   starts with a '-'.  This will ignore attempts to pass an argument
292   to "ls" that some clients try - it probably won't do what they 
293   expect, but at least they'll get a list of files.
294
295 2000-03-19, Shane Kerr <shane@time-travellers.org>
296 - Set file descriptors 0, 1, and 2 to go to "/dev/null", so that
297   any error messages sent by, say, the kernel don't accidentally
298   go to a user.
299 - Changed watchdog to have a single thread to watch all connections, 
300   rather than a thread per connection.  This is considerably more
301   complex, but effectively doubles the number of connections that
302   can be supported (due to thread limits).
303
304 2000-03-16, Shane Kerr <shane@time-travellers.org>
305 - Added support for REST command.  (Note that this is *not* the REST
306   exactly as described by RFC.  The REST in the RFC only applies to
307   block or compressed mode transfer, which oftpd does not currently
308   support.  However, it appears that Unix systems interpret the 
309   parameter to the REST command as the offset into the file to resume
310   from.
311
312 2000-03-13, Shane Kerr <shane@time-travellers.org>
313 - Version 0.1.3 released
314
315 2000-03-12, Shane Kerr <shane@time-travellers.org>
316 - Fixed bug when connection limit reached
317 - Fixed bug when attempt to bind() an already bound port
318
319 2000-03-12, Shane Kerr <shane@time-travellers.org>
320 - Version 0.1.2 released
321
322 2000-03-11, Shane Kerr <shane@time-travellers.org>
323 - Move configuration values into oftpd.h
324 - Wrap source to 80 columns
325 - Fix pthread_create() error check (code had incorrectly used errno
326   rather than the return of pthread_create() to determine error)
327 - Fix threads to run in detached mode (when appropriate)
328 - Solaris port completed
329 - Support for STOR added (reply with 553 error)
330 - Added free list for per-thread information structures.
331