fix compilation on solaris

[spaam]

When quotactl was introduced into libT/platform.c it failed to compile on solaris.

[taem]

Hi,

Thanks for your patch. On lines 17, 872 please use

#elif defined(__sun)

With

#elif __sun

compiler reports warnings that __sun is not defined on systems other than Solaris.

In getquota(), file handle 'fd' should be closed before exiting from function.

[spaam]

Fixed that and added some starting { to the other if statements so we can avoid some ifdefs.

[jordan]

Patch applied in r13975.

I changed the "#if sun" on line 17 to "#if defined (sun)" because the former generated an undefined reference warning on non-sun boxen.

Thanks for the patch!

[taem]

Hi spaam,

According to ​http://uw714doc.sco.com/en/man/html.4/mnttab.4.html, in your patch, on line 752 should be returned 'mnt_special' instead of 'mnt_fstype'?

And on line 799 should be used 'mnt_special' instead of 'mnt_mountp'?

Thanks.

[godot]

Revised patch on top of r14119

[godot]

The fix for this problem did not work for me. I got core dumps on solaris 2.11 using a binary compiled with 64bit ABI.

On Solaris, getment does not provide memory for the mntent structure itself. You have to allocate it yourself and pass an appropriate pointer to getmntent. See ​http://docs.oracle.com/cd/E19082-01/819-2243/getmntent-3c/index.html

This makes checking the return values cumbersome. I have thus remove the corresponding checks from the "return" line.

[jordan]

Patch applied in r14120. Thanks!