tooltalk: Fix bad assumptions about sizeof(uid_t)

In part of the tooltalk rpc code (mp_message.c), it was assumed that on
the majority of platforms, sizeof(uid_t)=sizeof(gid_t)=sizeof(long).  On
Linux-x64, uid_t is an unsigned int, which makes the code fail: all
tooltalk messages fail to send with an RPC_CANTENCODEARGS at the
rpc-level, and TT_INTERNAL_ERR for the actual program.  We instead
change the code to explicitly examine sizeof(uid_t) to see whether it is
int or long sized. This allows tooltalk-dependent functinoality
like logout and multiple calls to dtfile to work.

tt (tooltalk): Kill some warnings

Includes some potentially bad pointer/int conversions

tttrace: Fix bad usage of va_arg with enums

Enums may be represented with a smaller type than int; however, they are
automatically promoted to int when passed in va_arg lists, just as
short, char, etc. are. GCC thus "knows" that you never want to call
va_arg with an enum type, and instead inserts an abort.

Patches for dtscreen, dtsearchpath, dtsession on OBSD.

Do not redefine round(3), and provide a manpath for OpenBSD.
(this is one of the few things that need to be adjusted for other BSDs)

OpenBSD patches for ttsnoop.

Mostly adding std:: for strstream interfaces.

OpenBSD patches for dtpdmd and dtprintinfo.

Casts, #ifdefs, SIGCLD ...

OpenBSD #ifdefs for imake templates for localisations.

Make nsgmls compile on OpenBSD.

As far as I can tell, the duplicate instantiations from entmgr_inst.m4 are
unnecessary and only cause compile failures without -fpermissive.

Current state of my dtmail work.

Mostly #ifdefs and casts; also, do not redefine strcasestr().  This will
probably be needed for Linux too when compiling without -fpermissive.

Make dtlogin compile on OpenBSD.

Most importantly, we *cannot* do the utmp stuff this code is attempting.
It is SysV-specific.

Make dtinfo work on OpenBSD.

strstream.h is now called "strstream" and is obsolete, but use it anyway until
all code is converted over.  This also needs std:: added, at least for GCC
4.2.1.  Lastly, when hardcoding the path to perl, /usr/bin/perl should be used
rather than anything else.

OpenBSD patches for dtcalc, dtdocbook, dthelp and dtimsstart.

dtcreate: Fix double free inside ProcessExecString()

dtcreate: fix exit with TT_ERR_PTYPE and fix several sprintf related segfaults.

re-enable building of dtmail, except on OpenBSD

Imakefile diffs for OpenBSD.

imake: Convert function prototypes and signatures to ANSI format.

Fixes for dtfile on OpenBSD, plus missing prototypes. Use statfs() on BSD to find out if a file system is NFS.

OpenBSD fixes for dtcm. There are no global "timezone" and "tzname" symbols on BSD. Apart from that, mainly #ifdefs.

OpenBSD fixes for DtWidget, dtaction, dtappbuilder.

Allow dtterm to at least allocate a pty on OpenBSD. Display is still mangled, however, and it frequently crashes (64bit issue).

OpenBSD fixes for lib/DtTerm: values.h, #ifdef's and some constants that are not defined on BSD. Also, leave out utmp stuff that does not work on BSD. Note that this terminal allocation does not work properly yet, but this will be fixed later.

historical: mv some old files that we want to preserve out of the top level

Some of these older files just clutter up things and do not contribute
much information that is not historical in nature, so preserve them
in an out of the way location.

Tooltalk fixes for OpenBSD. This consists mainly of #ifdefs, casts and some small type nits.

OpenBSD fixes for DtPrint and csa.

New top level README from Christopher Turkel <turkelchris@aol.com>

dtwm: Cleanup some implicit definitions

Mostly this is adding appropriate #includes and declarations,
but for WmImage.c we also change from using the proper name
for XmeGetMask, rather than the identical but renamed
version _DtGetMask which is not exported in any header.

OpenBSD fixed for lib/DtSvc.

SIGPWR is not present everywhere.

OpenBSD.cf, from Pascal Stumpf

Provide support for installation under OpenBSD. Don't override PATH in the install script, it may hide needed tools, and the location of these binaries is highly OS-dependent anyway.

Also, allow overriding some variables in the install script.

recognise OpenBSD version

Rename getline() to not conflict with the standard POSIX interface of the same name.

Remove ambiguity in string compare.

Remove unnecessary extern modifier on struct declarations.

"ISO C++ forbids declaration of [X] with no type" warning fixes.

Merge branch 'master' of ssh://git.code.sf.net/p/cdesktopenv/code

Use socklen_t where available.

Patch from <Pascal.Stumpf@cubes.de>

This should use socklen_t where available, really ...

Use SIGCHLD rather than SIGCLD.

Patch from Pascal Stumpf <Pascal.Stumpf@cubes.de>

The official POSIX name for this signal is SIGCHLD.  Linux probably
has SIGCLD only for SysV compatibility, but BSD does not.

At least on OpenBSD/amd64, this macro returns a bogus value if its argument is NULL.

Patch from Pascal Stumpf <Pascal.Stumpf@cubes.de>

So check for this first like the other macros in this file do, and
don't cast an istring to int.

Deal with the fact that modern compilers assume different scoping rules

Patch from Pascal Stumpf <Pascal.Stumpf@cubes.de>:

So here are all the patches that deal with the fact that modern
compilers assume different scoping rules for variables declared in for
loops.  On Linux, -fpermissive has been added as a compiler flag to
compensate for this old C code, but I think it is the wrong approach.

Sorry, couldn't help sneaking in a || defined(CSRG_BASED) and some casts
needed for other reasons ...

This adds the basic config bits needed for OpenBSD.

Patch from Pascal Stumpf <Pascal.Stumpf@cubes.de>

Most is just copied from the existing imake installation in
/usr/X11R6.  Additionally:

* Allow overriding CDESharedRev, X11ProjectRoot and ProjectRoot
* Add a new define to be able to override MINCLUDESRC too

(this allows me to build CDE without symlinking any include directories)

Fix more sprintf calls.

Build with debug symbols enabled.

Use a more robust idiom When converting sprintf() to snprintf()

don't use the idiom

char foo[BUFSIZ];
snprintf(foo, BUFSIZ, ....);

but

char foo[BUFSIZ];
snprintf(foo, sizeo foo, ....);

because this will automatically catch situations where the size of foo
is later changed, e.g. like  foo[BUFSIZ + 8];

Fix another use of sprintf.

Fix for dtmail execution group error

dtscreen: include stdlib to avoid exit redeclaration

Signed-off-by: Robert Tomsick <robert@tomsick.net>

dtscreen: include stdlib where appropriate

Signed-off-by: Robert Tomsick <robert@tomsick.net>

dthelpview: Util.c include stdlib to avoid redeclaring exit()

Signed-off-by: Robert Tomsick <robert@tomsick.net>

dthelpview: Main.c - Cleanup variable init

Signed-off-by: Robert Tomsick <robert@tomsick.net>

dtcalc: warning prevention, add <stdlib.h> include for prototype of free()

dtcalc: warning prevention, do not use NULL when you mean 0.

dtaction: Fix unsafe use of sprintf

Patch from Robert Tomsick <robert+cde@tomsick.net>:

I believe this fixes vulnerability #3 from CERT CA-1999-11.[1]  The other
uses of sprintf in DtAction seem to be safe.

[1] https://www.cert.org/advisories/CA-1999-11.html

dtterm - minor warning fixes

Patch by Robert Tomsick <robert+cde@tomsick.net>

DtSvc: Fixes a segfault on Debian squeeze 64 bit and most probably other systems, too.

Patch from Marc Balmer <marc@msys.ch>:

Use strlen, not sizeof, here.  Fixes a segfault on Debian squeeze 64 bit
and most probably other systems, too.

libtt: "%wc" is Microsoft extension, not supported in every Std C Library.

Patch from Douglas Mencken <dougmencken@gmail.com>:

"%wc" is Microsoft extension, not supported in every Std C Library. So
if we don't want to print "%wc%wc%wc%wc%..." instead of real chars, we
shall not use it.

Before:

%wc%wc%wc%wc%wc%wc%wc%wc%wc%wc%wc%wc%wc%wc%wcession[28326]:
_Tt_s_session::s_init(): 1051 (TT_ERR_INTERNAL)!
%wc%wc%wc%wc%wc%wc%wc%wc%wc%wc%wc%wc%wc%wc%wcession[28326]: waitpid():
No child processes
%wc%wc%wc%wc%wc%wc%wc%wc%wc%wc%wc%wc%wc%wc%wcession[28326]: child
ttsession exited with status 1

After:

/usr/dt/bin/ttsession[12397]: _Tt_s_session::s_init(): 1051 (TT_ERR_INTERNAL)!
/usr/dt/bin/ttsession[12397]: waitpid(): No child processes
/usr/dt/bin/ttsession[12397]: child ttsession exited with status 1

DtSvc/dtwm: Fix some implicit declarations of functions by adding appropriate

Patch from Frederic Koehler <f.koehler427@gmail.com>:

These implicit definitions cause segfaults on x64 because
the implicit return type is a 32-bit signed int, rather than a pointer
type.

dtwm: Define a final fallback for loading default window manager font

Patch from Frederic Koehler <f.koehler427@gmail.com>:

Define a final fallback for loading default window manager font;
before exiting, forcefully try to load "fixed" font. This is sufficient
to allow systems where fontList is set to an empty list to startup dtwm,
for now.

libtt: Avoid an infinite loop in ttsession (tooltalk daemon) when /etc/mtab is a symlink, using lstat instead of stat.

Patch from Frederic Koehler <f.koehler427@gmail.com>

dtlogin: don't try to use /dev/mem to get random data on linux/bsd

dtlogin's genauth routines were trying to open and read /dev/mem on
linux and (presumably) bsd systems in order to obtain random data used
in creating an auth key.

This is bad for a variety of reasons.  Newer linux kernels (at least
on 3.2) issue the following warning to the kernel logs:

"Program dtlogin tried to access /dev/mem between 100000->102000."

Now on linux we will use /dev/urandom, and on CSRG_BASED (bsd) systems
we will use /dev/random to obtain some entropy.

dtdbcache: remove incorrect comment block and tmpnam_buf var (not used)

With Aaron's fixes to dtdbcache fixing a potential coredump, the
comment block in the write_db() function regarding tmpnam() no longer
applies, and the tmpnam_buf variable is no longer used.

So, remove them :)

dtdbcache: Remove old code in comments

Patch from Aaron W. Hsu <arcfide@sacrideo.us>

dtdbcache: Use mkstemp() instead of tmpnam() to avoid bad interactions

Patch from Aaron W. Hsu <arcfide@sacrideo.us>

Fix SIGSEGV while compiling dthelp on 64 bit systems.

Patch from Ulrich Wilkens <mail@uwilkens.de>

I have a little patch for a problem that I found when I tried to
compile dthelp on 64bit FreeBSD. It could also be a problem on other
64bit systems.  The problem is that the program context compiles but
fails running with segmentation fault.

context uses the function m_malloc() which is missing a correct
prototype sometimes. Then it's treated to return int instead of void *
. On 64bit systems this cuts off the higher 32 bits because void * is
64bit whereas int is only 32bit.

Imake.cf/linux.cf: Add arm support.

Patch from IRC user wschaub.

dtprintinfo: Fix SIGSEGV in dtprintinfo (passing NULL as first argument to catgets).

Patch from Douglas Mencken <dougmencken@gmail.com>:

The issue is that MESSAGE tries to invoke catgets with NULL first
parameter, which is dereferenced inside catgets (Std C Library
function) without checking, from catgets.c source:

if (catalog->name_ptr[idx + 0] == (u_int32_t) set
          && catalog->name_ptr[idx + 1] == (u_int32_t) message)

On the other hand, there's a special value: -1 (cast to nl_catd),
which must be used instead of 0 (NULL) in the case when we are unable
to provide real catalog_desc, from catgets.c source:

  /* Be generous if catalog which failed to be open is used.  */
  if (catalog_desc == (nl_catd) -1 || ++set <= 0 || message < 0)
    return (char *) string;

Enable PowerPC to be recognized by build.

Patch from Douglas Mencken.

Add Matthew Howkins (dtappbuilder, nsgmls, dtcm Linux compilation)

Also note that other contributors are listed in copyright and copyright.old

programs/Imakefile: don't try to build dtksh or dtinfo under linux.

dtlogin: Fix up Options->Language menu

In dtlogin, you can select the language to switch to by selecting it
via Options->Language.  Unfortunately this was also including '.' and
'..', since this list is built dynamically by scanning a directory.

Now we screen out '.' and '..'.

gitignore: add rest of generated files. git status is much cleaner now :)

dtbuilder: fixup TOG copyrights issue in generated files.

The TOG copyrights were being removed after a rebuild, leaving behind
the scary "RESTRICTED" copyright text that was originally there.

The issue was that the TOG copyrights were not properly embedded
within a 'DTB_USER_CODE_START' code block.

dtcodegen does not preserve any code outside DT_USER_CODE START and
END blocks.

Additionally, these objects are built with -merge by dtcodgen, so the
existing 'RESTRICTED' header within the codeblocks was being retained.

Merge branch 'master' of https://git.code.sf.net/p/cdesktopenv/code

Re-enable dtappbuilder on linux builds.

lnxLib.tmpl: remove bogus '#define NoMessageCatalog'

It is amazing what mayhem can be caused by a bad define :)

The reason most of the CDE programs were not actually using their
localized catalogs was because their use was being disabled by this
line.

Certain programs like dtwm and dthelp/libDtHelp did their own
localization handling and did not use the results of this define.
This is why they worked properly, and most everything else did not :)

linux.cf: do not bother using X_LOCALE at all.  We do not need it on linux.

libtt: client init(): use fcntl F_SETFD correctly.

Merge branch 'master' of https://git.code.sf.net/p/cdesktopenv/code

Allow installing the terminfo file for dtterm to also work with path as
/usr/share/terminfo as well as /usr/share/lib/terminfo

This allows fully functioning dtterm on Ubuntu 12.04 on which the terminfo file
failed to install.

dtwm: add missing workspace 'Occupy" commands to the C, de, es, fr, and it locales.

The dtwm DefaultWindowMenu did not list any of the workspace enabled
Occupy commands, which makes it a bit difficult to move windows into a
workspace, or to occupy multiple workspaces.

This was rectified by copying the relevant lines from the
SampleWindowMenu sections containing them that already existed in the
localized files.  Tested on C and DE (German).

Dtwm.defs.src: set proper mail file path for linux

copy CONTRIBUTORS file into /usr/dt/ during install

Merge branch 'master' of ssh://git.code.sf.net/p/cdesktopenv/code

CDE-MIN: copy the 'COPYING' file to /usr/dt/ on install

localized/templates: remove remaining attempts at building dtinfo related catalogs and other files

Attempts were still being made to build dtinfo.cat and other related
dtinfo files.  This could cause build failures in
programs/localized/...

sys.resources.src: make HIGH_COLOR the default

Add in a CONTRIBUTORS file for all people that have made changes post
open-source release. Also update sourceforge url to project website.

Commit the template changes for the locales including the missing msg files.

Add missing Mrm, Xm, and Uil .msg files for the various locales from the motif src.

Re-enable building them in the Imakefiles.

Merge branch 'master' of ssh://git.code.sf.net/p/cdesktopenv/code

Increment CDE version number to 2.2.0

instant: change warning message to fatal message, since it is.

instant: partially revert commit bc96e6f1ff6c72d4d2cafb6a4088ffa32cd3019f. remove 'HACK' :)

instant should fail if the locale xlation db cannot be opened.

The reason this was failing was because the locale translation DB for
linux did not exist (Linux.lcx).  I didn't add it until weeks/months
later :)

There needs to be one for each platform called "$(uname -s).lcx".
FreeBSD will need one too.

dtmail: IO.C: just compare against \r on all systems

Fix some minor issues and re-enable building of DE, ES, FR, and IT locale data (help and message cats)

Note, there are still some issues with certain programs appearing not
to use their localized message catalogs properly, while others work
fine.

Also, in order to build these now, you need to make sure you have
installed the DE, ES, FR, and IT locales before building or you will
get failures.  On [k]ubuntu, at least, you can install these with the
following commands:

sudo locale-gen de_DE
sudo locale-gen es_ES
sudo locale-gen fr_FR
sudo locale-gen it_IT

libDtHelp: fix up some warnigs and potential overruns

DtSvc/DtUtil1/Saver.c - fix the screen savers.

I have no idea how their putenv code can work.  Anyway, use code that
works on linux now. :)

dtsession: implement screen lock/unlock for linux

In order for this to work, dtsession must be setuid root.  If
dtsession is not setuid root, then locking will be disabled, and a
message will be written to ~/.dt/errorlog with the message: "Unable to
lock display due to security restrictions".

dtsession: SmUI.c: fix ancient typo

dtsession: also block HUP and PIPE when preparing to shutdown