| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com> |
| Cc: | pgsql-committers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: pgsql: Move SHA2 routines to a new generic API layer for crypto hashes |
| Date: | 2020-12-02 03:14:33 |
| Message-ID: | X8cGmZV/DIIFmptx@paquier.xyz |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
On Wed, Dec 02, 2020 at 11:42:58AM +0900, Fujii Masao wrote:
> On 2020/12/02 10:41, Michael Paquier wrote:
>> Move SHA2 routines to a new generic API layer for crypto hashes
>
> Thanks for improving this!
> I got the following compiler failure. ISTM that the variable "status"
> in both pg_cryptohash_update() and pg_cryptohash_final() needs
> to be initialized with 0. Patch attached. Thought?
Neither gcc nor clang complain on my end, even with
-Werror=maybe-uninitialized, because they understand that the status
will be set anyway as we have no default in the switch? :)
You are right, this is bad practice for those code paths, so let's fix
it. Which version are you using? I'd like to be able to reproduce
that and double-check if there are more missing spots.
--
Michael
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Michael Paquier | 2020-12-02 03:21:16 | Re: pgsql: Move SHA2 routines to a new generic API layer for crypto hashes |
| Previous Message | Fujii Masao | 2020-12-02 02:42:58 | Re: pgsql: Move SHA2 routines to a new generic API layer for crypto hashes |