Bearnip Lars LDMud LDMud 3.3 LDMud 3.3.720

LDMud 3.3.720

Version: 3.3.720 Release Date: %date%

Downloads:

• ldmud-3.3.720.tar.gz (MD5Sum)
• svn://svn.bearnip.com/ldmud/version/3.3.720

Summary of Changes

• Fixed Crashes
• xml_generate(), xml_parse() could crash the driver due to a wrong order during initialization of libxml2 (#671) and could lead to memory corruption, because the memory used by libxml2 was subject to the garbage collector and would be free'd by it while still in use.
• xml_generate(), xml_parse() could also crash the driver due to a wrong order during cleaning up, where a buffer could be freed prematurely.
• get_type_info() lost refcounts to the function name of a closure, causing it eventually being free'd prematurely (#712).
• 'dumpallobj' crashed if there are strings with 0 refcounts (#725).
• fixed a crash during tracing caused by a wrong call to add_message() in do_trace(), when compiled with --enable-debug (#736).
• Entering destructed objects as keys into a mapping could crash the driver later on.
• Very long function names with symbol_function() could crash the driver due to a corrupted stack.
• exec() when the player's socket buffer was full crashed.
• Destroying objects during initialisation, process_string() or transfer() could crash the driver.
• Changed Efuns:
  • get_dir():prepend '/' in plain mode for entries in the root directory, if paths are requested by GETDIR_PATH.
  • tail():marked as deprecated (#228)
  • sort_array() will sort in-place instead of a copying the array first, if the array is passed as reference. (#224)
  • to_struct() can convert a struct into another one if one of them is a (indirect) base of the other. (#646)
  • clone_object() does not deactivate an active heart_beat in a blueprint. (#747)
  • Fixed result of convert_charset():returns now the unchanged string if it gets an empty string for conversion. (#739)
  • Runtime
  • A base struct is recorded as base struct in a child struct even if the base struct has no members. (#695)
  • Improved the determination of the drivers current_time which fixes issues with wrongly timed heart_beats or apparantly wrong timings. (e.g. #743)
  • Migrated from signal() to sigaction() and alarm() to setitimer() to set the alarm and its handler, which seems to fix some issue with lost alarm signals. (#607)
  • Host system time and mud-internal time are now better synchronized.
  • Language
  • A new modifier 'deprecated' can be used to mark lfuns, sefuns and global variable as deprecated. Using/Calling them will cause warnings. (#749)
  • Networking
  • UDP datagrams with lengths up to 65507 (including UDP and IPv4 headers: 65535) bytes are supported.
  • SSLv2 connections are rejected, since they are insecure and long obsolete.
  • Some reasons for SSL disconnects were removed. (#737)
  • Configuration option tls_trustdirectory is deactivated by default.
  • All unknown telnet options are now passed to the H_TELNET_NEG hook. (#670)

Changelog

22-Feb-2011 (LDMud Team) 3.3.720

• Created snapshot. 2011-02-22 (Gnomi)
• Added checks for destroyed objects in clone_object(), process_string() and transfer(). (fixes #772) (simulate.c, efuns.c, object.c) 2011-02-19 (Zesstra)
• Check 'deprecated' modifier for global variables. In addition to functions the compiler and symbol_variable() check if a global variable is deprecated and issue a warning. Local variables are not checked. (closure.c, prolang.y, interpret.c, prolang.y) 2011-02-19 (Zesstra)
• Added check for 'deprecated' for lfun closures. Added a check for deprecated lfuns upon creation of closures by #'fun or #'::fun (compile-time). (lex.c, prolang.c) 2011-02-19 (Zesstra)
• New modifier 'deprecated' for lfuns. Functions can be marked with 'deprecated', which causes a warning when they are called. Intra-object calls will cause compile-time warnings, inter-objects calls (callother, lfun closure calls) compile-time warnings. (prolang.y, lex.c, interpret.c, closure.c) 2011-02-19 (Zesstra)
• Added testcases for some basic operators dealing with ints. Added some testcases for bitwise AND, OR, XOR, left shift, right shift, logical right shift, modulus and their respective assignment operators. 2011-02-19 (Zesstra)
• pkg-gnutls: added some attributes for nonnull pointer arguments. Some functions should never get NULL pointers as args. Marked these args as 'nonnull'. (pkg-gnutls.c) 2011-02-19 (Zesstra)
• pkg-gnutls:tls_write(): added comment about retries. GnuTLS and OpenSSL require a specific behaviour in case of interupted or blocking sockets. This adds a comment about the specifics of GnuTLS. (pkg-gnutls.c) 2011-02-19 (Zesstra)
• pkg-gnutls:tls_read(): let caller retry reading later. If the internal retry counter in tls_read() is not sufficient and the error is still GNUTLS_E_INTERRUPTED or GNUTLS_E_AGAIN return EINTR or EWOULDBLOCK to get_message() which will try again later. (bug #737) (pkg-gnutls.c) 2011-02-19 (Zesstra)
• comm_send_buf(): Removed dead code. Removed code that was never executed. Additionally, don't increment the inet_packets counter if nothing was sent. (comm.c) 2011-02-19 (Zesstra)
• Added some function attributes and assert(). Some functions should never get NULL pointers as args. Lengths of 0 in SSL_write() lead to undefined behaviour. Should never happen and will be asserted now. (pkg-openssl.c) 2011-02-19 (Zesstra)
• Simplified error handling in tls_read/tls_write(). Combined cases for err==0 and err<0 into one if block. (bug #737) (pkg-openssl.c) 2011-02-19 (Zesstra)
• Dump error stack to debug logs on disconnects. If compiled with DEBUG, the OpenSSL error stack will be dumped into the debug log on disconnects in tls_read()/tls_write(). May help finding the reasons for disconnects. (bug #737) (pkg-openssl.c) 2011-02-19 (Zesstra)
• Simplified error handling in get_message(). Converted a bunch of if() into a switch and use strerror() to ouput meaningful error messages in the default case (bug #737). (comm.c) 2011-02-19 (Zesstra)
• Removed some disconnects in case of recoverable errors. Sockets reads might be interupted by signals and return EINTR. In this case it will just be retried later again. tls_read() may stumble upon SSL_ERROR_WANT_READ/_WRITE. In these cases it now sets errno to EAGAIN instead of disconnecting the user. It also keeps the connection in case of SSL_ERROR_SYSCALL if errno is EAGAIN or EINTR (bug #737) (comm.c, pkg-openssl.c) 2011-02-19 (Zesstra)
• Set SSL_MODE_RELEASE_BUFFERS in SSL context. The flag configures OpenSSL to release unused buffers in idle SSL connections. (pkg-openssl.c) 2011-02-19 (Zesstra)
• Disallow SSLv2 connections. SSLv2 is long deprecated and should not be used anymore. (pkg-openssl.c) 2011-01-25 (Gnomi)
• Explicit context variables overwrote the types of the closure's arguments. (fixes #711) (prolang.y) 2011-01-25 (Gnomi)
• optpat() shouldn't read over the end of the given ed command. (fixes #722) (ed.c) 2011-01-11 (Zesstra)
• Add some fresh entropy to the PRNG from time to time. Everytime a new TLS connection is initialized, the current time (gettimeofday) is added to the entropy pool of OpenSSL. Additionally, for the next new connection after PRNG_RESEED_PERIOD/2 + random(PRNG_RESEED_PERIOD) seconds RAND_poll() from OpenSSL is called to add fresh entropy from a stronger source of entropy. (pkg-openssl.c) 2011-01-11 (Zesstra)
• Do not seed the OpenSSL PRNG with little entropy (#678) We used to seed OpenSSL's PRNG by using very weak sources of entropy. On system without a strong source of entropy (/dev/urandom, EGD etc.) this could lead to false sense of security. By not seeding ourselves, we force TLS to fail completely, if OpenSSL can't get enough entropy automagically. 2011-01-09 (Zesstra)
• Removed an error from varargs manpage (fixes #767). 2011-01-08 (Zesstra)
• _expand_define(): skip only leading/trailing whitespace. Commit 05202ac2b5789057de7f6d24e43fb59e969b4c12 skipped also the whitespace within the arguments, which has to be preserved. This commit changes the behaviour to only skip leading and trailing whitespace in the macro arguments. (fixes #729) (lex.c) 2011-01-06 (Zesstra)
• Fixed manpage of call_out. The manpage incorrectly stated a minimum delay of 1 second and mentioned rounding up to the next multiple of __ALARM_TIME__. Additionally added a comment about the real delay being between delay and delay+__ALARM_TIME__. 2011-01-06 (Zesstra)
• Rename IncCmdGiver to DecreaseCmdGiver. This define suggests to increment something, but it decrements. (comm.c) 2011-01-06 (Zesstra)
• _expand_define(): skip whitelist in argument list (fixes #729) _expand_define() skipped whitespace only at the start of the argument list, but not in between or trailing whitespace which lead to unexpected behaviour in case of #define quote(X) "X". (lex.c) 2011-01-05 (Zesstra)
• Set SSL_MODE_ACCEPT_MOVING_WRITE_BUFFER in SSL context. SSL_write() of OpenSSL requires to be called with the same arguments for a retry after SSL_WANT_READ/SSL_WANT_WRITE. With this flag, it also accepts a different buffer as long as its content is the same. Should solve some SSL/TLS disconnects (bug #737) (pkg-openssl.c) 2010-11-17 (Zesstra)
• Typo fix in m_contains manpage (fixes #761). 2010-11-17 (Zesstra)
• Corrected example for regreplace usage (fixes #760). 2010-07-15 (Zesstra)
• Check refcounter before division in svalue_size(). Reference counters are all checked to be != 0 before divisions, because the counters might overflow to 0 and might be constantly 0 from then on because reference counting is disabled for that entity. (dumpstat.c) 2010-07-14 (Zesstra)
• Changed signal handling in comm to sigaction(). Use sigaction() in comm.c as well. Removed the signal handler for SIGPIPE and ignore the signal (our old handler ignored it as well). (comm.c) 2010-07-14 (Zesstra)
• Synchronize the alarm timer to full seconds. As the mud internal time is only updated each alarm time this resulted in up to (alarm_time+1) seconds delay between the two. When the first timer is set up, we now take care to expire it on a full second to improve synchronicity (to a max. difference of (alarm_time)s). (backend.c) 2010-07-14 (Zesstra)
• Changed from alarm() to setitimer(ITIMER_REAL,) The alarm had to be (re-)set every time after an alarm occurred and if an alarm was missed somehow, this could lead to switched off alarms (main reason for check_alarm). We now use setitimer(ITIMER_REAL,...) to set a recurring realtime alarm, which simplifies matters and should make the check_alarm() obsolete. (backend.c) 2010-07-14 (Zesstra)
• Use sigaction for installing signal handlers. Because signal() can not be used in portable way to install signal handlers but sigaction() can, the driver installs the handlers now with sigaction(). Another advantage is, that the handler don't have to re-install themselves. (main.c, backend.c) 2010-06-03 (Zesstra)
• Added note about host system dependency for convert_charset(). The capabilities of convert_charset() depend on the libiconv of the host system. The manpage now has a hint about this. 2010-05-12 (Gnomi)
• Corrected the call to mysql_options. Use the generic error message when mysql returned an empty one. (pkg-mysql.c) 2010-05-04 (Fuchur)
• Enable automatic reconnects on MySQL database handles. (#750) 2010-05-01 (Zesstra)
• Updated settings for Avalon. 2010-04-26 (Zesstra)
• Fixed result of convert_charset(). If convert_charset() got an empty string for conversion, it returned its 3rd argument (to-charset). It now returns the unchanged string. (#739) (strfuns.c) 2010-04-26 (Zesstra)
• Don't deactivate a blueprints heartbeart upon cloning. The blueprints HB were supposedly deactivated upon cloning to prevent copying the active HB state to the new object. But since this anyway does not happen and clone_object() should not deactivate the blueprints HB permanently, this deactivation was removed (#747). (simulate.c) 2010-04-21 (Zesstra)
• Changed mk-patchlevel to create correct revision data. mk-patchlevel uses information from version.sh to add version and revision information to patchlevel.h. Unfortunately, version.sh seems to be updated by some unknown release script and contains horribly outdated information. mk-patchlevel tries now to extract the current revision from either SVN or git first and then tries version.sh. 2010-04-21 (Zesstra)
• Fixed manpages of load_name concerning VC objects. load_name() returns the original loadname of the object created by the VC. 2010-03-16 (Zesstra)
• Removed indefinite retries in tls_write(). tls_write() had no retry counter and retried for as long as the error was SSL_ERROR_WANT_READ/_WRITE. Similar to pkg-gnutls.c we just set errno to EWOULDBLOCK and let the caller handle the retries. (pkg-openssl.c) 2010-03-16 (Zesstra)
• Fixed return value of tls_read/tls_write(). A wrong usage of the variable ret could in some error conditions lead to a wrong return value and return a positive number of bytes read/written where nothing was read/written. (pkg-openssl.c) 2010-03-11 (Zesstra)
• Fixed add_message() call in do_trace(). do_trace() called add_message("%s",message_flush), which should be add_message(message_flush); Otherwise add_message() calls strlen() with the NULL pointer, leading to crashes (#736). (interpret.c) 2010-02-13 (Zesstra)
• Fixed Div-by-Zero in svalue_size(). svalue_size() divides the memory used for a string by its refcount, but 0 refs are valid for strings. This leads to a crash in then wiz command 'dumpallobj', unless the refcount is checked before division. (#725). (dumpstat.c) 2010-01-31 (Zesstra)
• Increased UDP receive buffer, because UDP datagrams can be much larger than 1024 bytes and the rest of the datagram would be discarded upon recvfrom(). Datagram lengths up to 65507 bytes (including UDP and IPv4 headers: 65535) are supported now. (comm.c)
• Removed dirty BeOS hack. (comm.c) 2010-01-24 (Zesstra)
• get_type_info() failed to reference the function name it returns, causing the string to loose refcounts and being eventually free'd erronenously. (#712) (efuns.c)
• Removed a potential double free in struct_new_anonymous(). (structs.c)
• Record a base struct as base struct in a child struct even if the base struct has no members. (#695) (prolang.y)
• to_struct() converts a given struct into a struct with the same type as the template struct given as optional second argument. (#646) This conversion is only done if the template struct is a base of the given struct or vice versa. Otherwise an error is raised. (efuns.c)
• Renamed baseof() to struct_baseof(). (structs.c, interpret.c, efuns.c)
• Use shrink_array() only for alists. Intersecting empty arrays or mappings references the null_vector instead of creating a new empty array with shrink_array(vec,0). shrink_array is only used in intersect_ordered_arr(). (array.c, mapping.c)

22-Dec-2009 (Gnomi)

• When removing unnecessary F_SAVE_ARG_FRAMEs the compiler didn't adjust last_expression correctly in same cases leading to false program optimizations. (#683, #708, #709) (prolang.y)

10-Nov-2009 (Gnomi)

• Check for references to destructed objects before entering them as keys into a mapping. (#689) (mapping.c)
• Pass all unknown telnet options to the mudlib. (#670) (comm.c)
• Let the player_for_flush list point to interactive_t* directly. This avoids inconsistencies between objects and their interactive structure caused when exec()ing. (Bug #683) (comm.c)

22-Oct-2009 (Zesstra)

• sort_array(): sort in-place if array is given as reference. (#224) (array.c, func_spec)

21-Oct-2009 (Zesstra)

• Applied a bunch of small fixes for manpages by Coogan. (#692)
• Marked tail as deprecated, added sefun tail(), updated documentation of tail(). (#228)

18-Oct-2009 (Gnomi)

• Free the xml buffer after xml reader and writer, because they might use it when cleaning up. (maybe #683) (pkg-xml2.c)
• Don't call xmlCleanupParser, as it should be called when there's no more xml processing. Don't call xmlInitParser, as it is already done on startup. (pkg-xml2.c)

13-Oct-2009 (Gnomi)

• Deactivate tls_trustdirectory per default as /etc/ssl/certs may contain a large number of certificates on usual installations (which are not related to client certificates) and GnuTLS clients have a problem with that (too large handshake data). Whoever wants to use client certificates shall set it up explicitely. (configure)
• Updated Avalon settings file. (settings/avalon)

05-Oct-2009 (Zesstra)

• prepend '/' in get_dir() in plain mode for entries in the root directory, if paths are requested by GETDIR_PATH. They were omitted until now. (#501) (files.c)

04-Oct-2009 (Zesstra)

• Use CRYPTO_set_mem_functions() to register function pointers to our own allocator functions in openssl. (pkg-openssl.c)

16-Sep-2009 (Zesstra)

• 'status/malloc' commands include now the allocator in use (Patch from Xtian, #672) (slaballoc.c, xptmalloc.c, smalloc.c) 12-Sep 2009 (Fuchur)
• fix some refcounting issues (interpret.c) 08-Sep 2009 (Zesstra)
• Get permanently allocated memory in xml_pkg_strdup(), which is not subject to the GC to prevent some nasty problems. (pkg-xml2.c)

04-Sep-2009 (Zesstra)

• Fixed initialization of libxml2 (#671). LIBXML_TEST_VERSION already allocates memory, so xmlMemSetup() has to be called before that macro. Otherwise crashes may occur. (pkg-xml2.c)
• Use the 'permanent' variants of our allocation function, because the memory used by libxml2 should not be subject to the garbage collector. (pkg-xml2.c, pkg-iksemel.c)

01-Sep-2009 (Gnomi)

• Fixed a crash in symbol_function with very long function names by removing the necessity of an alloca and giving make_shared_identifier the length of the identifier as an additional parameter. (lex.c, prolang.y)

03-Aug-2009 (Gnomi)

• Memory corruption when using protected string ranges. (interpret.c)

20-Jul-2009 (Gnomi)

• Set a maximum number of retries for reading data (in case the underlying functions go crazy and cause an endless loop). (pkg-gnutls.c, pkg-openssl.c)

23-Jun-2009 (Gnomi)

• Corrected a memory leak in send_udp. (comm.c)
• String corruption by present_clone. (efuns.c)

15-Jun-2009 (Fuchur)

• (parse.c) Fix off-by-one error in find_string(). It returned a wrong index.

Bearnip Lars LDMud LDMud 3.3 LDMud 3.3.720

Lars Düning () - PGP Key