Issue 23704008: Properly handle zero-sized fields in the PReg parser.

Issue 23704008: Properly handle zero-sized fields in the PReg parser. (Closed)

Created:
7 years, 3 months ago by Mattias Nissler (ping if slow)

Modified:
7 years, 3 months ago

Reviewers:
Joao da Silva

CC:
chromium-reviews, cpu_(ooo_6.6-7.5)

Base URL:
http://git.chromium.org/chromium/src.git@master

Visibility:
Public.

More Reviews

Description

Properly handle zero-sized fields in the PReg parser. For zero-sized values, we'd previously pass NULL to std::copy, which is not OK though if the number of elements to copy is 0. Avoid the copy altogether and cover this situation in the unit test. BUG=None R=joaodasilva@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=223345

Patch Set 1 #

Total comments: 2

Created: 7 years, 3 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+8 lines, -5 lines)			Patch
M	chrome/browser/policy/preg_parser_win.cc	View	1 chunk	+5 lines, -4 lines	2 comments	Download
M	chrome/browser/policy/preg_parser_win_unittest.cc	View	2 chunks	+4 lines, -2 lines	0 comments	Download
M	chrome/test/data/policy/registry.pol	View	0 chunks	+-1 lines, --1 lines	0 comments	Download

Messages

Total messages: 7 (0 generated)

Expand Messages | Collapse Messages

Mattias Nissler (ping if slow)

For reference, here's the new version of the registry.pol file, I just added one entry ...

7 years, 3 months ago (2013-09-13 13:52:24 UTC) #2

For reference, here's the new version of the registry.pol file, I just added one
entry in the end:

0000000: 5052 6567 0100 0000 5b00 5300 6f00 6600  PReg....[.S.o.f.
0000010: 7400 7700 6100 7200 6500 5c00 5000 6f00  t.w.a.r.e.\.P.o.
0000020: 6c00 6900 6300 6900 6500 7300 5c00 4300  l.i.c.i.e.s.\.C.
0000030: 6800 7200 6f00 6d00 6900 7500 6d00 0000  h.r.o.m.i.u.m...
0000040: 3b00 5300 6800 6f00 7700 4800 6f00 6d00  ;.S.h.o.w.H.o.m.
0000050: 6500 4200 7500 7400 7400 6f00 6e00 0000  e.B.u.t.t.o.n...
0000060: 3b00 0400 0000 3b00 0400 0000 3b00 0100  ;.....;.....;...
0000070: 0000 5d00 5b00 5300 6f00 6600 7400 7700  ..].[.S.o.f.t.w.
0000080: 6100 7200 6500 5c00 5000 6f00 6c00 6900  a.r.e.\.P.o.l.i.
0000090: 6300 6900 6500 7300 5c00 4300 6800 7200  c.i.e.s.\.C.h.r.
00000a0: 6f00 6d00 6900 7500 6d00 0000 3b00 4800  o.m.i.u.m...;.H.
00000b0: 6f00 6d00 6500 7000 6100 6700 6500 4c00  o.m.e.p.a.g.e.L.
00000c0: 6f00 6300 6100 7400 6900 6f00 6e00 0000  o.c.a.t.i.o.n...
00000d0: 3b00 0100 0000 3b00 2e00 0000 3b00 6800  ;.....;.....;.h.
00000e0: 7400 7400 7000 3a00 2f00 2f00 7700 7700  t.t.p.:././.w.w.
00000f0: 7700 2e00 6500 7800 6100 6d00 7000 6c00  w...e.x.a.m.p.l.
0000100: 6500 2e00 6300 6f00 6d00 0000 5d00 5b00  e...c.o.m...].[.
0000110: 5300 6f00 6600 7400 7700 6100 7200 6500  S.o.f.t.w.a.r.e.
0000120: 5c00 5000 6f00 6c00 6900 6300 6900 6500  \.P.o.l.i.c.i.e.
0000130: 7300 5c00 4300 6800 7200 6f00 6d00 6900  s.\.C.h.r.o.m.i.
0000140: 7500 6d00 0000 3b00 4800 6f00 6d00 6500  u.m...;.H.o.m.e.
0000150: 7000 6100 6700 6500 4900 7300 4e00 6500  p.a.g.e.I.s.N.e.
0000160: 7700 5400 6100 6200 5000 6100 6700 6500  w.T.a.b.P.a.g.e.
0000170: 0000 3b00 0400 0000 3b00 0400 0000 3b00  ..;.....;.....;.
0000180: 0100 0000 5d00 5b00 5300 6f00 6600 7400  ....].[.S.o.f.t.
0000190: 7700 6100 7200 6500 5c00 5000 6f00 6c00  w.a.r.e.\.P.o.l.
00001a0: 6900 6300 6900 6500 7300 5c00 4300 6800  i.c.i.e.s.\.C.h.
00001b0: 7200 6f00 6d00 6900 7500 6d00 0000 3b00  r.o.m.i.u.m...;.
00001c0: 5200 6500 7300 7400 6f00 7200 6500 4f00  R.e.s.t.o.r.e.O.
00001d0: 6e00 5300 7400 6100 7200 7400 7500 7000  n.S.t.a.r.t.u.p.
00001e0: 0000 3b00 0400 0000 3b00 0400 0000 3b00  ..;.....;.....;.
00001f0: 0400 0000 5d00 5b00 5300 6f00 6600 7400  ....].[.S.o.f.t.
0000200: 7700 6100 7200 6500 5c00 5000 6f00 6c00  w.a.r.e.\.P.o.l.
0000210: 6900 6300 6900 6500 7300 5c00 4300 6800  i.c.i.e.s.\.C.h.
0000220: 7200 6f00 6d00 6900 7500 6d00 5c00 5200  r.o.m.i.u.m.\.R.
0000230: 6500 7300 7400 6f00 7200 6500 4f00 6e00  e.s.t.o.r.e.O.n.
0000240: 5300 7400 6100 7200 7400 7500 7000 5500  S.t.a.r.t.u.p.U.
0000250: 5200 4c00 7300 0000 3b00 2a00 2a00 6400  R.L.s...;.*.*.d.
0000260: 6500 6c00 7600 6100 6c00 7300 2e00 0000  e.l.v.a.l.s.....
0000270: 3b00 0100 0000 3b00 0400 0000 3b00 2000  ;.....;.....;. .
0000280: 0000 5d00 5b00 5300 6f00 6600 7400 7700  ..].[.S.o.f.t.w.
0000290: 6100 7200 6500 5c00 5000 6f00 6c00 6900  a.r.e.\.P.o.l.i.
00002a0: 6300 6900 6500 7300 5c00 4300 6800 7200  c.i.e.s.\.C.h.r.
00002b0: 6f00 6d00 6900 7500 6d00 5c00 5200 6500  o.m.i.u.m.\.R.e.
00002c0: 7300 7400 6f00 7200 6500 4f00 6e00 5300  s.t.o.r.e.O.n.S.
00002d0: 7400 6100 7200 7400 7500 7000 5500 5200  t.a.r.t.u.p.U.R.
00002e0: 4c00 7300 0000 3b00 3100 0000 3b00 0100  L.s...;.1...;...
00002f0: 0000 3b00 3000 0000 3b00 6800 7400 7400  ..;.0...;.h.t.t.
0000300: 7000 3a00 2f00 2f00 7700 7700 7700 2e00  p.:././.w.w.w...
0000310: 6300 6800 7200 6f00 6d00 6900 7500 6d00  c.h.r.o.m.i.u.m.
0000320: 2e00 6f00 7200 6700 0000 5d00 5b00 5300  ..o.r.g...].[.S.
0000330: 6f00 6600 7400 7700 6100 7200 6500 5c00  o.f.t.w.a.r.e.\.
0000340: 5000 6f00 6c00 6900 6300 6900 6500 7300  P.o.l.i.c.i.e.s.
0000350: 5c00 4300 6800 7200 6f00 6d00 6900 7500  \.C.h.r.o.m.i.u.
0000360: 6d00 5c00 5200 6500 7300 7400 6f00 7200  m.\.R.e.s.t.o.r.
0000370: 6500 4f00 6e00 5300 7400 6100 7200 7400  e.O.n.S.t.a.r.t.
0000380: 7500 7000 5500 5200 4c00 7300 0000 3b00  u.p.U.R.L.s...;.
0000390: 3200 0000 3b00 0100 0000 3b00 2e00 0000  2...;.....;.....
00003a0: 3b00 6800 7400 7400 7000 3a00 2f00 2f00  ;.h.t.t.p.:././.
00003b0: 7700 7700 7700 2e00 6500 7800 6100 6d00  w.w.w...e.x.a.m.
00003c0: 7000 6c00 6500 2e00 6300 6f00 6d00 0000  p.l.e...c.o.m...
00003d0: 5d00 5b00 5300 6f00 6600 7400 7700 6100  ].[.S.o.f.t.w.a.
00003e0: 7200 6500 5c00 5000 6f00 6c00 6900 6300  r.e.\.P.o.l.i.c.
00003f0: 6900 6500 7300 5c00 4300 6800 7200 6f00  i.e.s.\.C.h.r.o.
0000400: 6d00 6900 7500 6d00 0000 3b00 2a00 2a00  m.i.u.m...;.*.*.
0000410: 4400 6500 6c00 6500 7400 6500 5600 6100  D.e.l.e.t.e.V.a.
0000420: 6c00 7500 6500 7300 0000 3b00 0100 0000  l.u.e.s...;.....
0000430: 3b00 6c00 0000 3b00 4400 6500 6c00 6500  ;.l...;.D.e.l.e.
0000440: 7400 6500 5600 6100 6c00 7500 6500 7300  t.e.V.a.l.u.e.s.
0000450: 5400 6500 7300 7400 3100 3b00 4400 6500  T.e.s.t.1.;.D.e.
0000460: 6c00 6500 7400 6500 5600 6100 6c00 7500  l.e.t.e.V.a.l.u.
0000470: 6500 7300 5400 6500 7300 7400 3200 3b00  e.s.T.e.s.t.2.;.
0000480: 4400 6500 6c00 6500 7400 6500 5600 6100  D.e.l.e.t.e.V.a.
0000490: 6c00 7500 6500 7300 5400 6500 7300 7400  l.u.e.s.T.e.s.t.
00004a0: 3300 0000 5d00 5b00 5300 6f00 6600 7400  3...].[.S.o.f.t.
00004b0: 7700 6100 7200 6500 5c00 5000 6f00 6c00  w.a.r.e.\.P.o.l.
00004c0: 6900 6300 6900 6500 7300 5c00 4300 6800  i.c.i.e.s.\.C.h.
00004d0: 7200 6f00 6d00 6900 7500 6d00 0000 3b00  r.o.m.i.u.m...;.
00004e0: 2a00 2a00 4400 6500 6c00 6500 7400 6500  *.*.D.e.l.e.t.e.
00004f0: 4b00 6500 7900 7300 0000 3b00 0100 0000  K.e.y.s...;.....
0000500: 3b00 6000 0000 3b00 4400 6500 6c00 6500  ;.`...;.D.e.l.e.
0000510: 7400 6500 4b00 6500 7900 7300 5400 6500  t.e.K.e.y.s.T.e.
0000520: 7300 7400 3100 3b00 4400 6500 6c00 6500  s.t.1.;.D.e.l.e.
0000530: 7400 6500 4b00 6500 7900 7300 5400 6500  t.e.K.e.y.s.T.e.
0000540: 7300 7400 3200 3b00 4400 6500 6c00 6500  s.t.2.;.D.e.l.e.
0000550: 7400 6500 4b00 6500 7900 7300 5400 6500  t.e.K.e.y.s.T.e.
0000560: 7300 7400 3300 0000 5d00 5b00 5300 6f00  s.t.3...].[.S.o.
0000570: 6600 7400 7700 6100 7200 6500 5c00 5000  f.t.w.a.r.e.\.P.
0000580: 6f00 6c00 6900 6300 6900 6500 7300 5c00  o.l.i.c.i.e.s.\.
0000590: 4300 6800 7200 6f00 6d00 6900 7500 6d00  C.h.r.o.m.i.u.m.
00005a0: 0000 3b00 2a00 2a00 4400 6500 6c00 2e00  ..;.*.*.D.e.l...
00005b0: 4400 6500 6c00 5400 6500 7300 7400 0000  D.e.l.T.e.s.t...
00005c0: 5d00 5b00 5300 6f00 6600 7400 7700 6100  ].[.S.o.f.t.w.a.
00005d0: 7200 6500 5c00 5000 6f00 6c00 6900 6300  r.e.\.P.o.l.i.c.
00005e0: 6900 6500 7300 5c00 4300 6800 7200 6f00  i.e.s.\.C.h.r.o.
00005f0: 6d00 6900 7500 6d00 5c00 4400 6500 6c00  m.i.u.m.\.D.e.l.
0000600: 5600 6100 6c00 7300 5400 6500 7300 7400  V.a.l.s.T.e.s.t.
0000610: 0000 3b00 2a00 2a00 4400 6500 6c00 5600  ..;.*.*.D.e.l.V.
0000620: 6100 6c00 7300 0000 5d00 5b00 5300 6f00  a.l.s...].[.S.o.
0000630: 6600 7400 7700 6100 7200 6500 5c00 5000  f.t.w.a.r.e.\.P.
0000640: 6f00 6c00 6900 6300 6900 6500 7300 5c00  o.l.i.c.i.e.s.\.
0000650: 4300 6800 7200 6f00 6d00 6900 7500 6d00  C.h.r.o.m.i.u.m.
0000660: 0000 3b00 2a00 2a00 5300 6500 6300 7500  ..;.*.*.S.e.c.u.
0000670: 7200 6500 4b00 6500 7900 0000 3b00 0400  r.e.K.e.y...;...
0000680: 0000 3b00 0400 0000 3b00 0100 0000 5d00  ..;.....;.....].
0000690: 5b00 5300 6f00 6600 7400 7700 6100 7200  [.S.o.f.t.w.a.r.
00006a0: 6500 5c00 5000 6f00 6c00 6900 6300 6900  e.\.P.o.l.i.c.i.
00006b0: 6500 7300 5c00 4300 6800 7200 6f00 6d00  e.s.\.C.h.r.o.m.
00006c0: 6900 7500 6d00 0000 3b00 2a00 2a00 5300  i.u.m...;.*.*.S.
00006d0: 6f00 6600 7400 0000 3b00 0400 0000 3b00  o.f.t...;.....;.
00006e0: 0400 0000 3b00 0100 0000 5d00 5b00 5300  ....;.....].[.S.
00006f0: 6f00 6600 7400 7700 6100 7200 6500 5c00  o.f.t.w.a.r.e.\.
0000700: 5000 6f00 6c00 6900 6300 6900 6500 7300  P.o.l.i.c.i.e.s.
0000710: 5c00 4300 6800 7200 6f00 6d00 6900 7500  \.C.h.r.o.m.i.u.
0000720: 6d00 0000 3b00 2a00 2a00 6200 6100 6400  m...;.*.*.b.a.d.
0000730: 0000 3b00 0400 0000 3b00 0400 0000 3b00  ..;.....;.....;.
0000740: 0100 0000 5d00 5b00 5300 6f00 6600 7400  ....].[.S.o.f.t.
0000750: 7700 6100 7200 6500 5c00 5000 6f00 6c00  w.a.r.e.\.P.o.l.
0000760: 6900 6300 6900 6500 7300 5c00 4500 7800  i.c.i.e.s.\.E.x.
0000770: 6100 6d00 7000 6c00 6500 0000 3b00 4200  a.m.p.l.e...;.B.
0000780: 6f00 6f00 6c00 6500 6100 6e00 0000 3b00  o.o.l.e.a.n...;.
0000790: 0400 0000 3b00 0400 0000 3b00 0100 0000  ....;.....;.....
00007a0: 5d00 5b00 5300 6f00 6600 7400 7700 6100  ].[.S.o.f.t.w.a.
00007b0: 7200 6500 5c00 5000 6f00 6c00 6900 6300  r.e.\.P.o.l.i.c.
00007c0: 6900 6500 7300 5c00 4300 6800 7200 6f00  i.e.s.\.C.h.r.o.
00007d0: 6d00 6900 7500 6d00 0000 3b00 5300 6e00  m.i.u.m...;.S.n.
00007e0: 6f00 7700 6d00 6100 6e00 0000 3b00 0100  o.w.m.a.n...;...
00007f0: 0000 3b00 0400 0000 3b00 0326 0000 5d00  ..;.....;..&..].
0000800: 5b00 5300 6f00 6600 7400 7700 6100 7200  [.S.o.f.t.w.a.r.
0000810: 6500 5c00 5000 6f00 6c00 6900 6300 6900  e.\.P.o.l.i.c.i.
0000820: 6500 7300 5c00 4300 6800 7200 6f00 6d00  e.s.\.C.h.r.o.m.
0000830: 6900 7500 6d00 0000 3b00 4500 6d00 7000  i.u.m...;.E.m.p.
0000840: 7400 7900 0000 3b00 0100 0000 3b00 0000  t.y...;.....;...
0000850: 0000 3b00 5d00                           ..;.].

Joao da Silva

lgtm with a suggestion https://chromiumcodereview-hr.appspot.com/23704008/diff/1/chrome/browser/policy/preg_parser_win.cc File chrome/browser/policy/preg_parser_win.cc (right): https://chromiumcodereview-hr.appspot.com/23704008/diff/1/chrome/browser/policy/preg_parser_win.cc#newcode73 chrome/browser/policy/preg_parser_win.cc:73: if (field_end <= *cursor || ...

7 years, 3 months ago (2013-09-13 14:18:07 UTC) #3

Mattias Nissler (ping if slow)

https://chromiumcodereview-hr.appspot.com/23704008/diff/1/chrome/browser/policy/preg_parser_win.cc File chrome/browser/policy/preg_parser_win.cc (right): https://chromiumcodereview-hr.appspot.com/23704008/diff/1/chrome/browser/policy/preg_parser_win.cc#newcode73 chrome/browser/policy/preg_parser_win.cc:73: if (field_end <= *cursor || field_end > end) On ...

7 years, 3 months ago (2013-09-13 14:40:24 UTC) #4

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mnissler@chromium.org/23704008/1

7 years, 3 months ago (2013-09-16 07:55:48 UTC) #5

commit-bot: I haz the power

Retried try job too often on win7_aura for step(s) unit_tests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&number=79211

7 years, 3 months ago (2013-09-16 09:57:51 UTC) #6

Mattias Nissler (ping if slow)

7 years, 3 months ago (2013-09-16 15:33:52 UTC) #7

Message was sent while issue was closed.

Committed patchset #1 manually as r223345 (presubmit successful).

Expand Messages | Collapse Messages