tree 16caabe753d594bf2a29782c9452e4aaa4efcfc0
parent fa7508de214c62dea9b485d1414d8b2a67bd8ef2
author yonghong-song <ys114321@gmail.com> 1529276112 -0700
committer GitHub <noreply@github.com> 1529276112 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJbJubQCRBK7hj4Ov3rIwAAdHIIACAZD6XttsOBoz2hhLUsjXsN
 JI/A05fO7IS9cURcWk9B+7+z4YW7MGCBD1q+VeOa3WA2ib38cDP1Yce7kNZXvGk2
 0Rx3zRvapwrIDcO5Ey3ZenY+vhfx6NZEUdOYr0ZnX1btSZQ/1/WSVUBxYntTBnHR
 ttmKOtg/Ek5ls5V+FFMorHtEPTFkf5ck0lpvKbfzre0YPpDHdLCvTfn32drtj9FA
 tYGDX6+FKzPDnvVcVVpA7yeiJvZo6vdmk8iiVYOP1D3dzCvIR1E35qGqXXgJ+YiE
 ClbTgovUaypmicvAnNR5ob+vzdCYNHiPHT54jj1JEgid/Nhf7MmnI7/WjcYTpro=
 =Cw/H
 -----END PGP SIGNATURE-----
 

avoid probe rewriting of p->m in &(p->m) (#1831)

Fix issue #1830.

After the rewrite, the code approximately becomes
  &({type _val; bpf_probe_read(&_val, sizeof(_val), &(p->m)); _val)

Firstly the rewriting is really unnecessary, and secondly
the compilation will fail since the addressOf cannot take address
of the rvalue _val.

C standard, however, allows the addressOf operand array subscript
expression, e.g.,
  &({type _val; bpf_probe_read(&_val, sizeof(_val), &(p->m)); _val)[0]

This patch intends to fix the problem by avoiding
the rewriting in the first place of addrressOf simple member expression.
It still permits addressOf the array subscript expression.

Signed-off-by: Yonghong Song <yhs@fb.com>