forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitd0071f9
committed
pgcrypto: Fix incorrect argument vs PG_GETARG*() mappings
The following functions use a mix of bytea and text arguments, but theirC internals always used PG_GETARG_BYTEA_PP(), creating an incorrect mixwith the argument types expected by encrypt_internal():- pgp_sym_encrypt_bytea(bytea,text[,text])- pgp_sym_encrypt(text,text[,text])- pgp_sym_decrypt_bytea(bytea,text[,text])- pgp_sym_decrypt(bytea,text[,text])- pgp_pub_encrypt_bytea(bytea,bytea[,text])- pgp_pub_encrypt(text,bytea[,text])- pgp_pub_decrypt_bytea(bytea, bytea[,text[,text]])- pgp_pub_decrypt(bytea,bytea[,text[,text]])This commit fixes the inconsistencies between the PG_GETARG*() macrosand the argument types of each function.Both BYTEA_PP() and TEXT_PP() rely on PG_DETOAST_DATUM_PACKED(), thatreturns an unaligned pointer, so this was not leading to an actualproblem as far as I know, but let's be consistent.Author: Shihao ZhongDiscussion:https://postgr.es/m/CAGRkXqRfiWT--DzVPx_UGpNHTt0YT5Jo8eV2CtT56jNP=QpXSQ@mail.gmail.com1 parentf854dae commitd0071f9
1 file changed
+29
-29
lines changedLines changed: 29 additions & 29 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
553 | 553 |
| |
554 | 554 |
| |
555 | 555 |
| |
556 |
| - | |
557 |
| - | |
| 556 | + | |
558 | 557 |
| |
559 |
| - | |
| 558 | + | |
| 559 | + | |
560 | 560 |
| |
561 | 561 |
| |
562 |
| - | |
| 562 | + | |
563 | 563 |
| |
564 |
| - | |
| 564 | + | |
565 | 565 |
| |
566 | 566 |
| |
567 | 567 |
| |
| |||
575 | 575 |
| |
576 | 576 |
| |
577 | 577 |
| |
578 |
| - | |
| 578 | + | |
579 | 579 |
| |
580 | 580 |
| |
581 | 581 |
| |
582 | 582 |
| |
583 |
| - | |
584 |
| - | |
| 583 | + | |
| 584 | + | |
585 | 585 |
| |
586 |
| - | |
| 586 | + | |
587 | 587 |
| |
588 | 588 |
| |
589 | 589 |
| |
| |||
598 | 598 |
| |
599 | 599 |
| |
600 | 600 |
| |
601 |
| - | |
602 |
| - | |
| 601 | + | |
603 | 602 |
| |
604 |
| - | |
| 603 | + | |
| 604 | + | |
605 | 605 |
| |
606 | 606 |
| |
607 |
| - | |
| 607 | + | |
608 | 608 |
| |
609 |
| - | |
| 609 | + | |
610 | 610 |
| |
611 | 611 |
| |
612 | 612 |
| |
| |||
620 | 620 |
| |
621 | 621 |
| |
622 | 622 |
| |
623 |
| - | |
624 |
| - | |
| 623 | + | |
625 | 624 |
| |
626 |
| - | |
| 625 | + | |
| 626 | + | |
627 | 627 |
| |
628 | 628 |
| |
629 |
| - | |
| 629 | + | |
630 | 630 |
| |
631 |
| - | |
| 631 | + | |
632 | 632 |
| |
633 | 633 |
| |
634 | 634 |
| |
| |||
654 | 654 |
| |
655 | 655 |
| |
656 | 656 |
| |
657 |
| - | |
| 657 | + | |
658 | 658 |
| |
659 | 659 |
| |
660 | 660 |
| |
| |||
668 | 668 |
| |
669 | 669 |
| |
670 | 670 |
| |
671 |
| - | |
672 |
| - | |
| 671 | + | |
673 | 672 |
| |
674 |
| - | |
| 673 | + | |
| 674 | + | |
675 | 675 |
| |
676 |
| - | |
| 676 | + | |
677 | 677 |
| |
678 | 678 |
| |
679 |
| - | |
| 679 | + | |
680 | 680 |
| |
681 | 681 |
| |
682 | 682 |
| |
| |||
700 | 700 |
| |
701 | 701 |
| |
702 | 702 |
| |
703 |
| - | |
| 703 | + | |
704 | 704 |
| |
705 |
| - | |
| 705 | + | |
706 | 706 |
| |
707 | 707 |
| |
708 | 708 |
| |
| |||
727 | 727 |
| |
728 | 728 |
| |
729 | 729 |
| |
730 |
| - | |
| 730 | + | |
731 | 731 |
| |
732 |
| - | |
| 732 | + | |
733 | 733 |
| |
734 | 734 |
| |
735 | 735 |
| |
|
0 commit comments
Comments
(0)