ViewVC Help
View File | Revision Log | Show Annotations | Download File
/cvs/CBOR-XS/XS.xs
(Generate patch)

Comparing CBOR-XS/XS.xs (file contents):
Revision 1.55 by root, Fri Nov 25 06:13:16 2016 UTC vs.
Revision 1.59 by root, Fri Nov 25 23:37:27 2016 UTC

… …
661 U32 depth; // recursion depth 661 U32 depth; // recursion depth
662 U32 maxdepth; // recursion depth limit 662 U32 maxdepth; // recursion depth limit
663 AV *shareable; 663 AV *shareable;
664 AV *stringref; 664 AV *stringref;
665 SV *decode_tagged; 665 SV *decode_tagged;
666 SV *err_sv; // optional sv for error, needs to be freed
666} dec_t; 667} dec_t;
667 668
668#define ERR(reason) SB if (!dec->err) dec->err = reason; goto fail; SE 669// set dec->err to ERRSV
670ecb_cold static void
671err_errsv (dec_t *dec)
672{
673 if (!dec->err)
674 {
675 dec->err_sv = newSVsv (ERRSV);
669 676
670#define WANT(len) if (ecb_expect_false (dec->cur + len > dec->end)) ERR ("unexpected end of CBOR data") 677 // chop off the trailing \n
678 SvCUR_set (dec->err_sv, SvCUR (dec->err_sv) - 1);
679 *SvEND (dec->err_sv) = 0;
671 680
681 dec->err = SvPVutf8_nolen (dec->err_sv);
682 }
683}
684
685// the following functions are used to reduce code size and help the compiler to optimise
686ecb_cold static void
687err_set (dec_t *dec, const char *reason)
688{
689 if (!dec->err)
690 dec->err = reason;
691}
692
693ecb_cold static void
694err_unexpected_end (dec_t *dec)
695{
696 err_set (dec, "unexpected end of CBOR data");
697}
698
699ecb_cold static void
700err_nesting_exceeded (dec_t *dec)
701{
702 err_set (dec, ERR_NESTING_EXCEEDED);
703}
704
705#define ERR_DO(do) SB do; goto fail; SE
706#define ERR(reason) ERR_DO (err_set (dec, reason))
707#define ERR_ERRSV ERR_DO (err_errsv (dec))
708
709#define WANT(len) if (ecb_expect_false ((UV)(dec->end - dec->cur) < (UV)len)) ERR_DO (err_unexpected_end (dec))
710
672#define DEC_INC_DEPTH if (++dec->depth > dec->cbor.max_depth) ERR (ERR_NESTING_EXCEEDED) 711#define DEC_INC_DEPTH if (ecb_expect_false (++dec->depth > dec->cbor.max_depth)) ERR (ERR_NESTING_EXCEEDED)
673#define DEC_DEC_DEPTH --dec->depth 712#define DEC_DEC_DEPTH --dec->depth
674 713
675static UV 714static UV
676decode_uint (dec_t *dec) 715decode_uint (dec_t *dec)
677{ 716{
… …
754 av_push (av, decode_sv (dec)); 793 av_push (av, decode_sv (dec));
755 } 794 }
756 } 795 }
757 else 796 else
758 { 797 {
759 int i, len = decode_uint (dec); 798 UV i, len = decode_uint (dec);
760 799
761 WANT (len); // complexity check for av_fill - need at least one byte per value, do not allow supersize arrays 800 WANT (len); // complexity check for av_fill - need at least one byte per value, do not allow supersize arrays
762 av_fill (av, len - 1); 801 av_fill (av, len - 1);
763 802
764 for (i = 0; i < len; ++i) 803 for (i = 0; i < len; ++i)
… …
781 // byte or utf-8 strings as keys, but only when !stringref 820 // byte or utf-8 strings as keys, but only when !stringref
782 821
783 if (ecb_expect_true (!dec->stringref)) 822 if (ecb_expect_true (!dec->stringref))
784 if (ecb_expect_true ((U8)(*dec->cur - MAJOR_BYTES) <= LENGTH_EXT8)) 823 if (ecb_expect_true ((U8)(*dec->cur - MAJOR_BYTES) <= LENGTH_EXT8))
785 { 824 {
786 I32 len = decode_uint (dec); 825 STRLEN len = decode_uint (dec);
787 char *key = (char *)dec->cur; 826 char *key = (char *)dec->cur;
788 827
789 WANT (len); 828 WANT (len);
790 dec->cur += len; 829 dec->cur += len;
791 830
… …
793 832
794 return; 833 return;
795 } 834 }
796 else if (ecb_expect_true ((U8)(*dec->cur - MAJOR_TEXT) <= LENGTH_EXT8)) 835 else if (ecb_expect_true ((U8)(*dec->cur - MAJOR_TEXT) <= LENGTH_EXT8))
797 { 836 {
798 I32 len = decode_uint (dec); 837 STRLEN len = decode_uint (dec);
799 char *key = (char *)dec->cur; 838 char *key = (char *)dec->cur;
800 839
801 WANT (len); 840 WANT (len);
802 dec->cur += len; 841 dec->cur += len;
803 842
… …
810 return; 849 return;
811 } 850 }
812 851
813 SV *k = decode_sv (dec); 852 SV *k = decode_sv (dec);
814 SV *v = decode_sv (dec); 853 SV *v = decode_sv (dec);
854
855 // we leak memory if uncaught exceptions are thrown by random magical
856 // methods, and this is hopefully the only place where it can happen,
857 // so if there is a chance of an exception, take the very slow path.
858 // since catching exceptions is "undocumented/internal/forbidden" by
859 // the new p5p powers, we need to call out to a perl function :/
860 if (ecb_expect_false (SvAMAGIC (k)))
861 {
862 dSP;
863
864 ENTER; SAVETMPS;
865 PUSHMARK (SP);
866 EXTEND (SP, 3);
867 PUSHs (sv_2mortal (newRV_inc ((SV *)hv)));
868 PUSHs (sv_2mortal (k));
869 PUSHs (sv_2mortal (v));
870
871 PUTBACK;
872 call_pv ("CBOR::XS::_hv_store", G_VOID | G_DISCARD | G_EVAL);
873 SPAGAIN;
874
875 FREETMPS; LEAVE;
876
877 if (SvTRUE (ERRSV))
878 ERR_ERRSV;
879
880 return;
881 }
815 882
816 hv_store_ent (hv, k, v, 0); 883 hv_store_ent (hv, k, v, 0);
817 SvREFCNT_dec (k); 884 SvREFCNT_dec (k);
818 885
819fail: 886fail:
… …
844 decode_he (dec, hv); 911 decode_he (dec, hv);
845 } 912 }
846 } 913 }
847 else 914 else
848 { 915 {
849 int pairs = decode_uint (dec); 916 UV pairs = decode_uint (dec);
917
918 WANT (pairs); // complexity check - need at least one byte per value, do not allow supersize hashes
850 919
851 while (pairs--) 920 while (pairs--)
852 decode_he (dec, hv); 921 decode_he (dec, hv);
853 } 922 }
854 923
… …
1050 SPAGAIN; 1119 SPAGAIN;
1051 1120
1052 if (SvTRUE (ERRSV)) 1121 if (SvTRUE (ERRSV))
1053 { 1122 {
1054 FREETMPS; LEAVE; 1123 FREETMPS; LEAVE;
1055 ERR (SvPVutf8_nolen (sv_2mortal (SvREFCNT_inc (ERRSV)))); 1124 ERR_ERRSV;
1056 } 1125 }
1057 1126
1058 SvREFCNT_dec (sv); 1127 SvREFCNT_dec (sv);
1059 sv = SvREFCNT_inc (POPs); 1128 sv = SvREFCNT_inc (POPs);
1060 1129
… …
1064 } 1133 }
1065 break; 1134 break;
1066 1135
1067 default: 1136 default:
1068 { 1137 {
1138 SV *tag_sv = newSVuv (tag);
1139
1069 sv = decode_sv (dec); 1140 sv = decode_sv (dec);
1070 1141
1071 dSP; 1142 dSP;
1072 ENTER; SAVETMPS; 1143 ENTER; SAVETMPS;
1073 SAVESTACK_POS (); 1144 SAVESTACK_POS ();
1074 PUSHMARK (SP); 1145 PUSHMARK (SP);
1075 EXTEND (SP, 2); 1146 EXTEND (SP, 2);
1076 PUSHs (newSVuv (tag)); 1147 PUSHs (tag_sv);
1077 PUSHs (sv); 1148 PUSHs (sv);
1078 1149
1079 PUTBACK; 1150 PUTBACK;
1080 int count = call_sv (dec->cbor.filter ? dec->cbor.filter : default_filter, G_ARRAY | G_EVAL); 1151 int count = call_sv (dec->cbor.filter ? dec->cbor.filter : default_filter, G_ARRAY | G_EVAL);
1081 SPAGAIN; 1152 SPAGAIN;
1082 1153
1083 if (SvTRUE (ERRSV)) 1154 if (SvTRUE (ERRSV))
1084 { 1155 {
1156 SvREFCNT_dec (tag_sv);
1085 FREETMPS; LEAVE; 1157 FREETMPS; LEAVE;
1086 ERR (SvPVutf8_nolen (sv_2mortal (SvREFCNT_inc (ERRSV)))); 1158 ERR_ERRSV;
1087 } 1159 }
1088 1160
1089 if (count) 1161 if (count)
1090 { 1162 {
1163 SvREFCNT_dec (tag_sv);
1091 SvREFCNT_dec (sv); 1164 SvREFCNT_dec (sv);
1092 sv = SvREFCNT_inc (POPs); 1165 sv = SvREFCNT_inc (POPs);
1093 } 1166 }
1094 else 1167 else
1095 { 1168 {
1096 AV *av = newAV (); 1169 AV *av = newAV ();
1097 av_push (av, newSVuv (tag)); 1170 av_push (av, tag_sv);
1098 av_push (av, sv); 1171 av_push (av, sv);
1099 1172
1100 HV *tagged_stash = !CBOR_SLOW || cbor_tagged_stash 1173 HV *tagged_stash = !CBOR_SLOW || cbor_tagged_stash
1101 ? cbor_tagged_stash 1174 ? cbor_tagged_stash
1102 : gv_stashpv ("CBOR::XS::Tagged" , 1); 1175 : gv_stashpv ("CBOR::XS::Tagged" , 1);
… …
1231 1304
1232 if (dec.err) 1305 if (dec.err)
1233 { 1306 {
1234 if (dec.shareable) 1307 if (dec.shareable)
1235 { 1308 {
1236 // need to break cyclic links, which whould all be in shareable 1309 // need to break cyclic links, which would all be in shareable
1237 int i; 1310 int i;
1238 SV **svp; 1311 SV **svp;
1239 1312
1240 for (i = av_len (dec.shareable) + 1; i--; ) 1313 for (i = av_len (dec.shareable) + 1; i--; )
1241 if ((svp = av_fetch (dec.shareable, i, 0))) 1314 if ((svp = av_fetch (dec.shareable, i, 0)))
1242 sv_setsv (*svp, &PL_sv_undef); 1315 sv_setsv (*svp, &PL_sv_undef);
1243 } 1316 }
1244 1317
1245 SvREFCNT_dec (sv); 1318 SvREFCNT_dec (sv);
1319
1320 if (dec.err_sv)
1321 sv_2mortal (dec.err_sv);
1322
1246 croak ("%s, at offset %d (octet 0x%02x)", dec.err, dec.cur - (U8 *)data, (int)(uint8_t)*dec.cur); 1323 croak ("%s, at offset %d (octet 0x%02x)", dec.err, dec.cur - (U8 *)data, (int)(uint8_t)*dec.cur);
1247 } 1324 }
1248 1325
1249 sv = sv_2mortal (sv); 1326 sv = sv_2mortal (sv);
1250 1327
… …
1394 1471
1395 default_filter = newSVpv ("CBOR::XS::default_filter", 0); 1472 default_filter = newSVpv ("CBOR::XS::default_filter", 0);
1396 1473
1397 sv_cbor = newSVpv ("CBOR", 0); 1474 sv_cbor = newSVpv ("CBOR", 0);
1398 SvREADONLY_on (sv_cbor); 1475 SvREADONLY_on (sv_cbor);
1476
1477 assert (("STRLEN must be an unsigned type", 0 <= (STRLEN)-1));
1399} 1478}
1400 1479
1401PROTOTYPES: DISABLE 1480PROTOTYPES: DISABLE
1402 1481
1403void CLONE (...) 1482void CLONE (...)

Diff Legend

– Removed lines
+ Added lines
< Changed lines
> Changed lines