2002-11-15 Theodore A. Roth * ChangeLog: * configure.in: * src/DAPA.C: Index: ChangeLog =================================================================== RCS file: /cvsroot/uisp/uisp/ChangeLog,v retrieving revision 1.28 diff -u -r1.28 ChangeLog --- ChangeLog 30 Oct 2002 23:22:06 -0000 1.28 +++ ChangeLog 15 Nov 2002 21:40:24 -0000 @@ -1,3 +1,18 @@ +2002-11-15 Theodore A. Roth + + * configure.in (AM_INIT_AUTOMAKE): Bump version. + * src/DAPA.C: Use /dev/parport0 as the default interface instead of + direct port access via ioperm(). Using uisp at root or setuid root is + frowned upon so it should not be the default. + +2002-11-15 Theodore A. Roth + (Thanks to Christopher X. Candreva for reporting this.) + + * src/Serial.h: Increase serial timeout. Programming 8535 parts with + the stk500 was timing out to early and caused failures. This also seems + to affect the at90s1200 devices (also reported by Klaus Rudolph + ). + 2002-10-30 Theodore A. Roth (Thanks to Seth LaForge for pointing out the buffer overflow problems.) Index: configure.in =================================================================== RCS file: /cvsroot/uisp/uisp/configure.in,v retrieving revision 1.14 diff -u -r1.14 configure.in --- configure.in 30 Oct 2002 23:22:07 -0000 1.14 +++ configure.in 15 Nov 2002 21:40:24 -0000 @@ -25,7 +25,7 @@ AC_INIT(src/Main.C) AC_CONFIG_AUX_DIR(config) AM_CONFIG_HEADER(src/config.h:src/config-h.in) -AM_INIT_AUTOMAKE(uisp, 20021030cvs) +AM_INIT_AUTOMAKE(uisp, 20021115cvs) dnl Checks for programs. CHECK_AUTOCONF213 Index: src/DAPA.C =================================================================== RCS file: /cvsroot/uisp/uisp/src/DAPA.C,v retrieving revision 1.6 diff -u -r1.6 DAPA.C --- src/DAPA.C 20 Aug 2002 04:50:30 -0000 1.6 +++ src/DAPA.C 15 Nov 2002 21:40:25 -0000 @@ -923,7 +923,14 @@ parport_base(0x378), ppdev_fd(-1) { const char *val; - const char *ppdev_name = NULL; + + /* If the user doesn't specify -dlpt option, use /dev/parport0 as the + default instead of defaulting to using ioperm (ick!). If the user wants + to run uisp as root (or setuid root) they should know what they are doing + and can suffer the consequences. Joe user should not be told about ioperm + failure due to permission denied. */ + + const char *ppdev_name = "/dev/parport0"; /* Enable Parallel Port */ val = GetCmdParam("-dprog"); @@ -982,8 +989,6 @@ #ifdef NO_DIRECT_IO if ((val = GetCmdParam("-dlpt")) != NULL) { ppdev_name = val; - } else { - ppdev_name = "/dev/parport0"; } #endif