Re: [HACKERS] acl problem in NetBSD/m68k

From: Bruce Momjian <maillist(at)candle(dot)pha(dot)pa(dot)us>
To: t-ishii(at)sra(dot)co(dot)jp
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, hackers(at)postgreSQL(dot)org
Subject: Re: [HACKERS] acl problem in NetBSD/m68k
Date: 1999-07-08 02:06:41
Message-ID: 199907080206.WAA19858@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


Did we ever fix this?

> >> grant/revoke does not work in NetBSD/m68k. This is due to the wrong
> >> assumption that sizeof(AclItem) is equal to 8 in all platforms. I am
> >> going to fix this by replacing all occurrence of sizeof(AclItem) to
> >> ACLITEM_SIZE (newly defined as 8 in catalog/pg_type.h). See included
> >> patches. If there's no objection, I will commit them. Comments?
> >
> >I do not like this patch at *all*. Why is sizeof(AclItem) not the
> >correct thing to use?
>
> In NetBSD/m68k sizeof(AclItem) = 6, not 8.
>
> >Replacing it with a hardwired "8" seems like
> >a step backwards --- not to mention a direct contradiction of what
> >you claim the patch is doing.
>
> It's already hard wired in pg_type.h, isn't it.
>
> >Perhaps the real problem is that the AclItem struct definition needs
> >modification? Or maybe we need a way to put a machine-dependent size
> >into the pg_type entry for type aclitem? The latter seems like a
> >good thing to be able to do on general principles.
>
> Glad to hear you have better idea. Anyway, NetBSD/m68k users need some
> way to fix the problem now, since the problem seems very serious.
> --
> Tatsuo Ishii
>
>

--
Bruce Momjian | http://www.op.net/~candle
maillist(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Hiroshi Inoue 1999-07-08 02:07:10 RE: [HACKERS] spinlock freeze again
Previous Message Bruce Momjian 1999-07-08 02:02:52 Re: [HACKERS] PATCH for pgconnection.h