Changes between Initial Version and Version 1 of Ticket #163, comment 10
- Timestamp:
- 05/06/13 13:21:29 (11 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #163, comment 10
initial v1 1 1 Thank you very much for your thorough review of this issue, Waffelman. Now it is fully possible to understand why pfs is NULL, which definitely was the most important thing here. 2 2 3 That it is NULL due to no parsed pf file, makes a lot of sense. So I agree, the patch can now be included. The other approach and writing to this file, to give it some valid contents seems like the wrong solution to me.3 That it is NULL due to no parsed pf contents (file or management interface), makes a lot of sense. So I agree, the patch can now be included. The other approach and writing to this file, to give it some valid contents seems like the wrong solution to me. 4 4 5 And I checked quickly the call paths for pf_cn_test(), which always called via pf_c2c_test(). So it seems to be something like this:5 And I checked quickly the call paths for pf_cn_test(), which always is called via pf_c2c_test(). So it seems to be something like this: 6 6 7 7 {{{